Skip to content

fix(ci): repair daily Update API Documentation workflow#5

Open
islandbitcoin wants to merge 1 commit into
mainfrom
fix/update-docs-generate-script
Open

fix(ci): repair daily Update API Documentation workflow#5
islandbitcoin wants to merge 1 commit into
mainfrom
fix/update-docs-generate-script

Conversation

@islandbitcoin

Copy link
Copy Markdown
Contributor

Problem

The daily Update API Documentation workflow has been failing on every run:

npm error Missing script: "generate-docs"

The Magidoc migration (#4) renamed the generate-docs npm script to generate and folded it into build (fetch-schema + magidoc generate + clean-urls). The workflow's Generate documentation step still called the removed npm run generate-docs, failing the job before docs could regenerate.

Fix

Remove the Generate documentation step. It was both broken (script gone) and redundant — the next step, Build site, already runs npm run build, which fetches the schema and regenerates the site.

No behavior change on success: schema is still fetched, change-detected, built, committed, and pushed.

🤖 Generated with Claude Code

The Magidoc migration (PR #4) renamed the 'generate-docs' npm script to
'generate' and folded it into 'build' (fetch-schema + generate +
clean-urls). The workflow still called 'npm run generate-docs', which no
longer exists, failing the daily 'Update API Documentation' run with
'Missing script: generate-docs'. The subsequent 'Build site' step already
runs 'npm run build' and covers generation, so the step was both broken
and redundant. Remove it.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01C7z7o9J18BWbUnYJcemMtg
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