You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Fix the MCP deployment so Wrangler uses Bun from its subdirectory.
Publish @ooxml-dev/cli@0.1.0 to npm and configure the release workflow to authenticate future npm releases.
Replace the root Bun devEngines declaration with packageManager so npm can pack and publish the CLI while Vite+ still selects Bun.
Update the README with the npm install command, the research-ooxml skill, and a short production-service sign-in note.
Keep MCP protocol details in the MCP section and remove auth implementation details from the main README.
Why
The CLI had a 0.1.0 tag but had never been published. npm also could not run because the root devEngines declaration required Bun. These changes publish the CLI and keep future releases automated through Changesets.
Verified
bun run check
bun run cli:test
bun run --cwd apps/cli build
npm publish --dry-run --access public
Published @ooxml-dev/cli@0.1.0 through release run 31688309173
Installed @ooxml-dev/cli@0.1.0 from npm and ran ooxml --version and ooxml --help
bun run wrangler deploy --dry-run from apps/mcp-server
The release workflow now depends on secrets.NPM_TOKEN being configured with publish permissions;
if it’s missing/invalid, the Changesets publish step will fail with an npm authentication error and
block releases.
ⓘ Recommendations generated based on similar findings in past PRs
Evidence
The workflow passes NODE_AUTH_TOKEN from secrets.NPM_TOKEN into the Changesets step, and the
invoked publish script is changeset publish, which requires npm authentication to publish
packages.
The issue below was found during a code review. Follow the provided context and guidance below and implement a solution
### Issue description
The workflow now publishes via `changeset publish` and relies on `NODE_AUTH_TOKEN` from `secrets.NPM_TOKEN`. If the secret is unset/invalid or lacks publish permission, the workflow will fail late during publish with an npm 401, blocking releases.
### Issue Context
`release:publish` runs `changeset publish`, which requires npm registry authentication.
### Fix Focus Areas
- .github/workflows/release.yml[31-46]
- package.json[20-24]
### Suggested fix
1) Add an explicit preflight step before `changesets/action` to validate that `secrets.NPM_TOKEN` is present (non-empty), and optionally that it can authenticate (e.g., `npm whoami`), failing with a clear message.
2) Ensure repo/org secrets documentation (or the workflow itself via `error::`) indicates the required secret name and required npm permissions (automation token that can publish `@ooxml-dev/*`).
ⓘ Copy this prompt and use it to remediate the issue with your preferred AI generation tools
Review mode: ⚖️ Balanced: This small workflow-only change alters npm release authentication and token propagation, a security- and release-sensitive path warranting a complete review.
Tip of the day
💡 Did you know, you can type 'qodo, fix this' on a finding and the fix lands right on your PR
Verified in the real release path: NPM_TOKEN is configured with publish access, and release run 31688309173 published @ooxml-dev/cli@0.1.0. A separate npm whoami step would not prove package-level publish access and could block version-PR creation when no npm publish is needed, so the existing npm error is the clearer failure path.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What changed
@ooxml-dev/cli@0.1.0to npm and configure the release workflow to authenticate future npm releases.devEnginesdeclaration withpackageManagerso npm can pack and publish the CLI while Vite+ still selects Bun.research-ooxmlskill, and a short production-service sign-in note.Why
The CLI had a
0.1.0tag but had never been published. npm also could not run because the rootdevEnginesdeclaration required Bun. These changes publish the CLI and keep future releases automated through Changesets.Verified
bun run checkbun run cli:testbun run --cwd apps/cli buildnpm publish --dry-run --access public@ooxml-dev/cli@0.1.0through release run 31688309173@ooxml-dev/cli@0.1.0from npm and ranooxml --versionandooxml --helpbun run wrangler deploy --dry-runfromapps/mcp-servernpx skills add superdoc/ooxml-dev --skill research-ooxml -g -y