Skip to content

Remove tracked dist artifacts, drop legacy TS tests/sources, and fix AJV import for tsc#11

Merged
GsCommand merged 1 commit into
mainfrom
codex/fix-runtime-core-pr-issues
May 12, 2026
Merged

Remove tracked dist artifacts, drop legacy TS tests/sources, and fix AJV import for tsc#11
GsCommand merged 1 commit into
mainfrom
codex/fix-runtime-core-pr-issues

Conversation

@GsCommand
Copy link
Copy Markdown
Contributor

Motivation

  • Remove generated dist/ artifacts from source control so build outputs are not tracked and future PRs remain clean.
  • Eliminate legacy files causing TypeScript compile/test drift and remove superseded test/source files.
  • Fix a TypeScript constructability error when dynamically importing ajv so npm run build and tsc succeed.

Description

  • Stopped tracking dist/ and added dist/ to .gitignore so distribution files are no longer included in the repo.
  • Deleted legacy files src/receipt-v1.ts and src/test/runtime-core.test.ts which were superseded by the modern sources and tests.
  • Updated src/schema-client.ts to cast the dynamically imported ajv default export to a constructor (AjvCtor) and instantiate it via new AjvCtor(...) to resolve the TS constructability error.
  • Did not add any src/shims.d.ts and left src/canonicalize.ts present as required.

Testing

  • Ran npm install which failed in this environment due to a registry 403 Forbidden error when fetching @types/node, so full install of dev/test deps did not complete.
  • Ran npm run build which succeeded after the ajv import/instantiation fix, resolving the earlier TS error.
  • Ran npm test which failed due to missing test runtime helper tsx/esm (transitively unavailable because npm install did not complete).
  • Ran npm run typecheck (tsc --noEmit) which passed with the applied changes.

Codex Task

@GsCommand GsCommand merged commit 7e27251 into main May 12, 2026
0 of 2 checks passed
@GsCommand GsCommand deleted the codex/fix-runtime-core-pr-issues branch May 13, 2026 01:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant