Skip to content

docs(spec): NIW2-312 upgrade hono and harden the release pipeline - #86

Closed
csdev19 wants to merge 1 commit into
mainfrom
feature/niw2-312-upgrade-hono-and-harden-the-dependency-and-release-pipeline
Closed

csdev19 wants to merge 1 commit into
mainfrom
feature/niw2-312-upgrade-hono-and-harden-the-dependency-and-release-pipeline

Conversation

@csdev19

@csdev19 csdev19 commented Oct 12, 2026

Copy link
Copy Markdown
Owner

Refs NIW2-312. Spec: design/supply-chain-hardening.md. ADRs: 0018 (dependency version policy), 0019 (what the release pipeline trusts).

Decisions

  • hono goes to ^4.13.13: 0 advisories, against 38 at 4.12.3.
  • Runtime dependencies use caret ranges and stay unbundled. Exact pins are banned in dependencies.
  • Dependabot, not Renovate, covers bun and github-actions, weekly and grouped.
  • Every action is pinned to a SHA, and the repository setting that requires SHA pins gets turned on.
  • npm Trusted Publishing replaces NPM_TOKEN, with Node 24 in the publish job. An ordered cut-over checklist keeps a release from breaking.
  • Release and deploy run behind a verify job on the tagged SHA, plus a production audit and a pack-and-install smoke test.
  • validate.yml runs on release-please--* branches and becomes a required check.
  • verify covers every workspace.
  • Dev dependencies are refreshed: vitest 4, oxfmt latest, sharp, the MCP SDK and vite.
  • Repository settings get hardened. The owner checklist is in the spec.

ADR-0019 supersedes the "Tokens" section of ADR-0010 and contradicts its addendum. The spec explains both.

Open questions

  1. If Dependabot ignores catalog: entries, switch to Renovate or bump those by hand? Recommendation: try Dependabot first.
  2. Add a required reviewer on production, at the cost of one click per release? Recommendation: yes.
  3. Vitest 4 now, or 5? Recommendation: 4.
  4. Upgrade oxfmt from 0.19 to 0.72, which costs one reflow commit? Recommendation: yes.
  5. Run audit in verify, or only in CI? Recommendation: CI only.
  6. Astro 5 to 7: recommendation is a separate ticket.

Status

This PR contains only the design spec and ADR drafts. It is opened as a draft so the design is not lost. The implementation will land on this same branch after the open questions are answered. Everything is "proposed, not implemented".

Part of the 2026-10-12 audit, umbrella NIW2-302. Specs were written by Fable agents against a read-only export of main at ee34302. The ADR numbers (0014–0026) are assigned across all eight audit PRs. decisions/index.md is not edited in these PRs, to avoid eight-way conflicts; add the index rows when each PR is promoted.

🤖 Generated with Claude Code

Spec and ADR drafts for NIW2-312, from the 2026-10-12 security, quality and
scalability audit (umbrella NIW2-302). Status: proposed, not implemented.

Refs: NIW2-312

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@csdev19 csdev19 closed this Oct 12, 2026
@csdev19
csdev19 deleted the feature/niw2-312-upgrade-hono-and-harden-the-dependency-and-release-pipeline branch October 12, 2026 17:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant