Repository navigation
Update Juno SDK files - #66
github-actions[bot] wants to merge 3 commits into
Conversation
Greptile OverviewGreptile SummaryThis PR appears to be an OpenAPI-generator refresh of the Juno SDK client code (API classes and model definitions), with large-scale reformatting and a reduced set of exported models/APIs. Key integration impact: There is also a concrete contract mismatch in Confidence Score: 2/5
Important Files Changed
|
| export * from './authApi'; | ||
| export * from './emailApi'; | ||
| export * from './fileBucketApi'; | ||
| export * from './fileConfigApi'; | ||
| export * from './fileDownloadApi'; | ||
| export * from './fileProviderApi'; | ||
| export * from './fileUploadApi'; | ||
| export * from './projectApi'; | ||
| export * from './userApi'; | ||
| import * as http from 'http'; | ||
| import { AnalyticsApi } from './analyticsApi'; | ||
| import { AnalyticsConfigApi } from './analyticsConfigApi'; | ||
| import { AuthApi } from './authApi'; | ||
| export * from './emailApi'; | ||
| import { EmailApi } from './emailApi'; | ||
| export * from './fileBucketApi'; | ||
| import { FileBucketApi } from './fileBucketApi'; | ||
| export * from './fileConfigApi'; | ||
| import { FileConfigApi } from './fileConfigApi'; | ||
| export * from './fileDownloadApi'; | ||
| import { FileDownloadApi } from './fileDownloadApi'; | ||
| export * from './fileProviderApi'; | ||
| import { FileProviderApi } from './fileProviderApi'; | ||
| export * from './fileUploadApi'; | ||
| import { FileUploadApi } from './fileUploadApi'; | ||
| export * from './projectApi'; | ||
| import { ProjectApi } from './projectApi'; | ||
| export * from './userApi'; | ||
| import { UserApi } from './userApi'; | ||
| import * as http from 'http'; | ||
|
|
||
| export class HttpError extends Error { | ||
| constructor( | ||
| public response: http.IncomingMessage, | ||
| public body: any, | ||
| public statusCode?: number | ||
| ) { | ||
| super('HTTP request failed'); | ||
| this.name = 'HttpError'; | ||
| } | ||
| constructor (public response: http.IncomingMessage, public body: any, public statusCode?: number) { | ||
| super('HTTP request failed'); | ||
| this.name = 'HttpError'; | ||
| } | ||
| } | ||
|
|
||
| export { RequestFile } from '../model/models'; | ||
|
|
||
| export const APIS = [ | ||
| AnalyticsApi, | ||
| AnalyticsConfigApi, | ||
| AuthApi, | ||
| EmailApi, | ||
| FileBucketApi, | ||
| FileConfigApi, | ||
| FileDownloadApi, | ||
| FileProviderApi, | ||
| FileUploadApi, | ||
| ProjectApi, | ||
| UserApi, | ||
| ]; | ||
| export const APIS = [AuthApi, EmailApi, FileBucketApi, FileConfigApi, FileDownloadApi, FileProviderApi, FileUploadApi, ProjectApi, UserApi]; |
There was a problem hiding this comment.
Removed analytics exports
apis.ts no longer exports or includes AnalyticsApi / AnalyticsConfigApi in APIS (compared to the previous version). If downstream code expects import { AnalyticsApi } from .../apis or iterates APIS to register all SDK APIs, this is a breaking change and will fail at compile-time/runtime. If analytics is still part of the SDK surface, restore the exports and add the classes back into APIS.
| /* tslint:disable:no-unused-locals */ | ||
| import { AggregateInterval } from '../model/aggregateInterval'; | ||
| import { EmailConfigResponse } from '../model/emailConfigResponse'; | ||
| import { RegisterDomainModel } from '../model/registerDomainModel'; | ||
| import { RegisterDomainResponse } from '../model/registerDomainResponse'; | ||
| import { RegisterEmailModel } from '../model/registerEmailModel'; | ||
| import { RegisterEmailResponse } from '../model/registerEmailResponse'; | ||
| import { SendEmailModel } from '../model/sendEmailModel'; | ||
| import { SendEmailResponse } from '../model/sendEmailResponse'; | ||
| import { SendEmailStatisticsResponses } from '../model/sendEmailStatisticsResponses'; | ||
| import { SetupEmailResponse } from '../model/setupEmailResponse'; | ||
| import { SetupEmailServiceModel } from '../model/setupEmailServiceModel'; | ||
| import { VerifyDomainModel } from '../model/verifyDomainModel'; | ||
|
|
||
| import { | ||
| ObjectSerializer, | ||
| Authentication, | ||
| VoidAuth, | ||
| Interceptor, | ||
| } from '../model/models'; | ||
| import { | ||
| HttpBasicAuth, | ||
| HttpBearerAuth, | ||
| ApiKeyAuth, | ||
| OAuth, | ||
| } from '../model/models'; | ||
| import { ObjectSerializer, Authentication, VoidAuth, Interceptor } from '../model/models'; | ||
| import { HttpBasicAuth, HttpBearerAuth, ApiKeyAuth, OAuth } from '../model/models'; |
There was a problem hiding this comment.
Removed email statistics types
EmailApi no longer imports/returns SendEmailStatisticsResponses (and AggregateInterval was also removed). If the /email/* endpoints still expose statistics in the OpenAPI spec, this change breaks the SDK surface (missing types + missing methods or changed return types). Please confirm the API spec actually removed these endpoints/types; otherwise restore the generated models and corresponding API methods.
| export * from './accessKey'; | ||
| export * from './aggregateInterval'; | ||
| export * from './analyticsConfigResponse'; | ||
| export * from './clickEventProperties'; | ||
| export * from './clickEventResponse'; | ||
| export * from './createAnalyticsConfigModel'; | ||
| export * from './createProjectModel'; | ||
| export * from './createUserModel'; | ||
| export * from './customEventResponse'; | ||
| export * from './customEventTypeResponse'; | ||
| export * from './customGraphType'; | ||
| export * from './customGraphTypeResponse'; | ||
| export * from './deleteFileBucketModel'; | ||
| export * from './downloadFileModel'; | ||
| export * from './downloadFileResponse'; | ||
| export * from './emailConfigResponse'; | ||
| export * from './emailContent'; | ||
| export * from './emailDomain'; | ||
| export * from './emailRecipient'; | ||
| export * from './emailSender'; | ||
| export * from './emailSenderSendEmailModel'; | ||
| export * from './fileBucket'; | ||
| export * from './fileConfigResponse'; | ||
| export * from './fileProvider'; | ||
| export * from './fileProviderPartial'; | ||
| export * from './getAllClickEventsResponse'; | ||
| export * from './getAllCustomEventTypeResponse'; | ||
| export * from './getAllCustomEventsResponse'; | ||
| export * from './getAllInputEventsResponse'; | ||
| export * from './getAllVisitEventsResponse'; | ||
| export * from './getClickEventsResponse'; | ||
| export * from './getCustomEventsResponse'; | ||
| export * from './getInputEventsResponse'; | ||
| export * from './getVisitEventsResponse'; | ||
| export * from './inputEventProperties'; | ||
| export * from './inputEventResponse'; | ||
| export * from './fileProviderResponse'; | ||
| export * from './issueApiKeyRequest'; | ||
| export * from './issueApiKeyResponse'; | ||
| export * from './issueJWTResponse'; | ||
| export * from './linkProjectModel'; | ||
| export * from './linkUserModel'; | ||
| export * from './logClickEventRequest'; | ||
| export * from './logCustomEventRequest'; | ||
| export * from './logInputEventRequest'; | ||
| export * from './logVisitEventRequest'; | ||
| export * from './modelFile'; | ||
| export * from './projectResponse'; | ||
| export * from './registerDomainModel'; | ||
| export * from './registerDomainResponse'; | ||
| export * from './registerEmailModel'; | ||
| export * from './registerEmailResponse'; | ||
| export * from './registerFileBucketModel'; | ||
| export * from './registerFileProviderModel'; | ||
| export * from './sendEmailModel'; | ||
| export * from './sendEmailResponse'; | ||
| export * from './sendEmailStatisticsResponse'; | ||
| export * from './sendEmailStatisticsResponses'; | ||
| export * from './sendGridDNSRecord'; | ||
| export * from './sendGridDNSResponse'; | ||
| export * from './setUserTypeModel'; | ||
| export * from './setupEmailResponse'; | ||
| export * from './setupEmailServiceModel'; | ||
| export * from './setupFileServiceResponse'; | ||
| export * from './unlinkProjectModel'; | ||
| export * from './updateAnalyticsConfigModel'; | ||
| export * from './uploadFileModel'; | ||
| export * from './uploadFileResponse'; | ||
| export * from './userResponse'; | ||
| export * from './userResponses'; | ||
| export * from './verifyDomainModel'; |
There was a problem hiding this comment.
Breaking SDK surface change
src/internal/model/models.ts dropped exports/imports for many previously-generated models (analytics/event logging, file provider types like FileProvider, unlink/update models, email statistics, etc.). Any consumer importing these from ../model/models will now fail to compile. If these endpoints/types still exist in the API, this indicates the SDK was regenerated against an incomplete/different OpenAPI spec; please regenerate with the correct spec (or restore the missing models/exports).
Additional Comments (2)
|
8c8579e to
eb67a4b
Compare
Update Juno SDK files