chore: migrate to the @j4k scope on the Forge npm registry - #29
Conversation
Rename devDep oxlint-config-j4k -> @j4k/oxlint-config (^2.11.0, the only version on the Forge) and run j4k-align --fix from current main: - pnpm-workspace.yaml registries flip to default npmjs + @j4k on code.j4k.dev, with minimumReleaseAgeIgnoreMissingTime false and an @j4k/* release-age exclude. - checks/dedupe/release-npm install auth via FORGEJO_NPM_TOKEN against code.j4k.dev; pr-review passes FORGEJO_NPM_TOKEN into the axgithub reusable's legacy NPM_TOKEN input. - oxlint ^1.72.0, oxfmt ^0.57.0, packageManager pnpm@11.8.0 floors. - lockfile rebuilt against the Forge (0 npm.j4k.dev refs, cold-store frozen install verified) + pnpm dedupe. knip entry widened to types/**/*.d.ts so the managed css-modules.d.ts shim is treated as an ambient entry point instead of an unused file. The repo-scoped NPM_TOKEN secret was deleted by --fix (forbidden post-flip); any pre-flip branch CI fails installs until main merges and the branch refreshes. pr-review draws stay red until the owner re-tags axgithub v1 (still the Verdaccio-era reusable); pr-review is not a required check.
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: f939423cc5
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
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".
| trap 'rm -f "$script_log"' EXIT | ||
|
|
||
| # Silent on success; on failure, dump captured output and exit non-zero. | ||
| # Logs a "==> <name>" header, then runs the script with output captured and shown only on failure before exiting non-zero. |
There was a problem hiding this comment.
Remove the WHAT-only hook comment
AGENTS.md's “Comments Explain Why, Not What” rule says to default to no comments and never explain what the code already shows. This added line only narrates the immediately following run_script body—echoing a header, capturing output, and exiting on failure—so it violates the repo's documented comment standard without adding non-obvious context; remove it or replace it with the hidden reason this wrapper exists.
Useful? React with 👍 / 👎.
Phase 4 (§1B) migration of the private-dep install path from Verdaccio
(
npm.j4k.dev) to the Forgejo npm registry (code.j4k.dev).What changed
oxlint-config-j4k->@j4k/oxlint-config(^2.11.0,the only version on the Forge). Managed
oxlint.config.tsimportre-rendered by
j4k-align --fix.pnpm-workspace.yamlregistries flip:default: https://registry.npmjs.org/+"@j4k": https://code.j4k.dev/api/packages/j4k/npm/, withminimumReleaseAgeIgnoreMissingTime: falseand an@j4k/*release-age exclude (the Forge serves no
timemetadata).checks/dedupe-check/release-npminstall auth is nowNODE_AUTH_TOKEN: secrets.FORGEJO_NPM_TOKENwriting acode.j4k.dev_authTokenline,each guarded by a missing-secret check.
pr-reviewpassesFORGEJO_NPM_TOKENinto theaxgithubreusable's legacyNPM_TOKENsecret input.
--fix:oxlint ^1.72.0,oxfmt ^0.57.0,packageManager pnpm@11.8.0; managedtypes/css-modules.d.tsshim,.githooks/pre-commit,.oxfmtrc.json,.gitignoreworktrees entry.@j4k/oxlint-config+pnpm dedupe.grep -c npm.j4k.dev pnpm-lock.yamlis0; a cold-storepnpm install --frozen-lockfile(fresh
--store-dir) fetched@j4k/oxlint-config@2.11.0from theForge and passed integrity.
entrywidened totypes/**/*.d.tsso the newly-managedcss-modules.d.tsambient shim is an entry point, not an unused file.Secret change (action needed on other open PRs)
--fixdeleted the repo-scopedNPM_TOKENActions secret (forbiddenpost-flip). Any pre-flip branch's CI will fail its install step until
this PR merges to
mainand that branch is refreshed withmain.FORGEJO_NPM_TOKENwas already provisioned on this repo.Known advisory breakage (do not chase)
pr-reviewdelegates toJercik/axgithub@v1, still the Verdaccio-erareusable. Passing
FORGEJO_NPM_TOKENto it authenticates against thewrong registry, so PR Review draws will be red until the owner
re-tags
axgithubv1(PRs #6/#7 pending).pr-reviewis not arequired status check.
Out of scope (pre-existing, advisory)
engines.node ">=24.0.0"is below the>=24.15.0audit floor and 7devDeps are behind latest — both advisory, unrelated to the registry
flip, and left untouched so this stays a non-releasing change.