Addresses two todo items

Author: tylerlong

packages/core/src/index.ts

@@ -231,7 +231,7 @@ class RingCentral implements RingCentralInterface {
     await this.restapi(null)
       .oauth()
       .revoke()
-      .post({ token: temp, client_id: this.rest.clientId } as any); // todo: spec doesn't allow client_id
+      .post({ token: temp, client_id: this.rest.clientId });
     this.token = undefined;
   }
 

packages/extensions/authorize-uri/src/index.ts

@@ -1,24 +1,13 @@
 import type RingCentral from "@rc-ex/core";
 import SdkExtension from "@rc-ex/core/SdkExtension";
-// import type AuthorizeRequest from "@rc-ex/core/definitions/AuthorizeRequest";
+import type AuthorizeRequest from "@rc-ex/core/definitions/AuthorizeRequest";
 import type { QueryDataMap } from "urijs";
 import URI from "urijs";
 
 export interface AuthorizeUriOptions {
   baseUri?: string;
 }
 
-// todo: remove this type when spec updated
-type AuthorizeRequest = {
-  response_type?: string;
-  client_id?: string;
-  code_challenge?: string;
-  code_challenge_method?: string;
-  redirect_uri?: string;
-  state?: string;
-  ui_options?: string;
-};
-
 function isNode() {
   return typeof process !== "undefined" &&
     process.versions?.node != null &&