Refactor authored tooling to TypeScript - #2
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Keep authored SDK, tests, operational scripts, and Next configuration in TypeScript while compiling Node-compatible JavaScript artifacts for execution. Constraint: Node 18 and framework loaders require compiled JavaScript for runtime entrypoints Rejected: TypeScript runtime loader dependency | adds runtime coupling and weakens the published package contract Confidence: high Scope-risk: moderate Directive: Keep authored application code in TypeScript; retain JavaScript only where a tool loader requires it or as compiler output Tested: pnpm check; pnpm --filter agent-compat-docs build; 53 tests and 41 adapter conformance cases passed Not-tested: Remote CI after this commit
Semantic-release refuses Node 20, so the release workflow must use Node 22 while the package runtime contract remains Node 18. Constraint: semantic-release requires Node 22.14 or newer Rejected: Downgrade semantic-release | would discard the current release toolchain Confidence: high Scope-risk: narrow Directive: Keep CI release Node aligned with semantic-release support requirements Tested: git diff --check; workflow change is configuration-only Not-tested: semantic-release execution locally because local Node is 20
💡 Codex Reviewagent-compat/packages/agents/package.json Lines 33 to 34 in 5b5d36d When the tag-triggered This re-export now exposes only the reduced set in ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍. Codex can also answer questions or update the PR. Try commenting "@codex address that feedback". |
Summary
Verification