Skip to content

[runtime] remove redundant keygen helper, add Dockerfile, clean smoke test#26

Merged
GsCommand merged 1 commit intomainfrom
codex/remove-duplicate-key-generation-utility
Mar 20, 2026
Merged

[runtime] remove redundant keygen helper, add Dockerfile, clean smoke test#26
GsCommand merged 1 commit intomainfrom
codex/remove-duplicate-key-generation-utility

Conversation

@GsCommand
Copy link
Copy Markdown
Contributor

Motivation

  • Consolidate runtime tooling onto the canonical key generator, provide a simple container entrypoint for the runtime service, and remove a dead smoke-test helper that encoded a stale fixed-kid assumption.

Description

  • Delete the redundant gen-ed25519.mjs and retain tools/mkkeys.mjs as the canonical keygen utility used by scripts/dev.sh and the README.
  • Add a minimal runtime Dockerfile based on node:20-bookworm-slim that runs npm ci --omit=dev, copies server.mjs, exposes port 8080, and starts the app with npm start.
  • Remove the unused assertReceiptInvariants() helper from tests/smoke.mjs so the smoke test no longer hardcodes kid === "v1" and instead relies on the active extractProof() assertions that validate the runtime's proof shape.

Testing

  • Ran npm run check (Node syntax check) which completed successfully.
  • Ran the smoke script with node tests/smoke.mjs, which completed successfully (smoke ok).
  • No other automated tests were modified.

Codex Task

Why: consolidate runtime tooling on the canonical key generator, add a simple runtime service container, and remove dead smoke-test receipt validation code with stale kid assumptions.

Contract impact: none
@GsCommand GsCommand merged commit 9c27517 into main Mar 20, 2026
1 check passed
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