Skip to content

chore(api): update generated SDK attribution - #2125

Open
apcha-oai wants to merge 1 commit into
mainfrom
castiron/promotions/pr-24-0f28f1a81a3e
Open

chore(api): update generated SDK attribution#2125
apcha-oai wants to merge 1 commit into
mainfrom
castiron/promotions/pr-24-0f28f1a81a3e

Conversation

@apcha-oai

Copy link
Copy Markdown
Contributor

Summary

chore(api): update generated SDK attribution

Castiron-Internal-PR: openai/openai-node-internal#24
Castiron-Source-SHA: 0f28f1a81a3ef6ee4d3fcb1e2ac1dd9b5ef418d7
Castiron-Public-Base-SHA: cf6237f
import { APIResource } from '../../core/resource';
import * as OrganizationAPI from './organization/organization';
import { Organization } from './organization/organization';
import { APIPromise } from '../../core/api-promise';
import * as OrganizationAPI from './organization/organization';
import { Organization } from './organization/organization';
import { APIPromise } from '../../core/api-promise';
import { RequestOptions } from '../../internal/request-options';
// File generated from our OpenAPI spec by Castiron. See CONTRIBUTING.md for details.

import { APIResource } from '../../../core/resource';
import { APIPromise } from '../../../core/api-promise';
UserUpdateParams,
Users,
} from './users/users';
import { APIPromise } from '../../../core/api-promise';
Users,
} from './users/users';
import { APIPromise } from '../../../core/api-promise';
import { RequestOptions } from '../../../internal/request-options';
// File generated from our OpenAPI spec by Castiron. See CONTRIBUTING.md for details.

import { APIResource } from '../../../../core/resource';
import { APIPromise } from '../../../../core/api-promise';
TranslationVerbose,
Translations,
} from './translations';
import { APIPromise } from '../../core/api-promise';
Translations,
} from './translations';
import { APIPromise } from '../../core/api-promise';
import { RequestOptions } from '../../internal/request-options';
ThreadListParams,
Threads,
} from './threads';
import { APIPromise } from '../../../core/api-promise';
Threads,
} from './threads';
import { APIPromise } from '../../../core/api-promise';
import { RequestOptions } from '../../../internal/request-options';
@apcha-oai
apcha-oai marked this pull request as ready for review August 7, 2026 03:16
@apcha-oai
apcha-oai requested a review from a team as a code owner August 7, 2026 03:16
@openai-sdks

openai-sdks Bot commented Aug 7, 2026

Copy link
Copy Markdown

OkTest Summary

237/237 SDK tests passed in 9.924s for Node SDK PR #2125.

Test results — 42 files
Test Result Time
tests/chat-completions-complex-body.test.ts ✅ Passed 135ms
tests/chat-completions-create.test.ts ✅ Passed 204ms
tests/chat-completions-stream.test.ts ✅ Passed 194ms
tests/files-content-binary.test.ts ✅ Passed 173ms
tests/files-create-multipart.test.ts ✅ Passed 121ms
tests/files-list-pagination.test.ts ✅ Passed 134ms
tests/initialize-config.test.ts ✅ Passed 152ms
tests/instance-isolation.test.ts ✅ Passed 92ms
tests/models-list.test.ts ✅ Passed 128ms
tests/responses-background-lifecycle.test.ts ✅ Passed 188ms
tests/responses-body-method-errors.test.ts ✅ Passed 357ms
tests/responses-cancel-timeout.test.ts ✅ Passed 230ms
tests/responses-cancel.test.ts ✅ Passed 220ms
tests/responses-compact-retries.test.ts ✅ Passed 275ms
tests/responses-compact.test.ts ✅ Passed 208ms
tests/responses-create-advanced-stream.test.ts ✅ Passed 109ms
tests/responses-create-advanced.test.ts ✅ Passed 156ms
tests/responses-create-disconnect.test.ts ✅ Passed 1.17s
tests/responses-create-errors.test.ts ✅ Passed 183ms
tests/responses-create-malformed-api-responses.test.ts ✅ Passed 195ms
tests/responses-create-retries.test.ts ✅ Passed 236ms
tests/responses-create-stream-failures.test.ts ✅ Passed 180ms
tests/responses-create-stream-timeout.test.ts ✅ Passed 2.151s
tests/responses-create-stream-wire.test.ts ✅ Passed 2.435s
tests/responses-create-stream.test.ts ✅ Passed 95ms
tests/responses-create-terminal-states.test.ts ✅ Passed 184ms
tests/responses-create-timeout.test.ts ✅ Passed 213ms
tests/responses-create.test.ts ✅ Passed 147ms
tests/responses-delete.test.ts ✅ Passed 218ms
tests/responses-input-items-errors.test.ts ✅ Passed 174ms
tests/responses-input-items-list.test.ts ✅ Passed 179ms
tests/responses-input-items-options.test.ts ✅ Passed 110ms
tests/responses-input-tokens-count-timeout.test.ts ✅ Passed 251ms
tests/responses-input-tokens-count.test.ts ✅ Passed 219ms
tests/responses-malformed-inputs.test.ts ✅ Passed 2.11s
tests/responses-not-found-errors.test.ts ✅ Passed 239ms
tests/responses-parse.test.ts ✅ Passed 173ms
tests/responses-retrieve-retries.test.ts ✅ Passed 181ms
tests/responses-retrieve.test.ts ✅ Passed 176ms
tests/responses-stored-method-errors.test.ts ✅ Passed 456ms
tests/retry-behavior.test.ts ✅ Passed 2.953s
tests/sdk-error-shape.test.ts ✅ Passed 256ms

View OkTest run #31143849878

SDK merge (3c17c4387c82) · head (36d1fc11909b) · base (cf6237f23ca1) · OkTest (91635c6a2723)

@apcha-oai
apcha-oai enabled auto-merge August 7, 2026 03:55

@jbeckwith-oai jbeckwith-oai left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This attribution-only regeneration also adds 41 unused imported symbols across 24 generated files (for example, src/resources/admin/admin.ts, src/resources/shared.ts, and src/resources/fine-tuning/methods.ts; the two Responses files also add an unused PagePromise). The generated-file lint exclusion hides the full set, although code-quality has already flagged representative instances. Please fix Castiron import collection to emit only referenced symbols and regenerate instead of hand-editing generated output. After ignoring the attribution headers, these imports are the only TypeScript changes, so the regenerated source should remain attribution-only.

@HAYDEN-OAI HAYDEN-OAI added the upstream Needs to be addressed in Stainless upstream label Aug 7, 2026 — with ChatGPT Codex Connector
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

upstream Needs to be addressed in Stainless upstream

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants