Skip to content

Commit f834e81

Browse files
PaulDuvallclaude
andcommitted
fix: update REQ-053 test to use simplified npm-publish-simple.yml workflow
- Change test to look for npm-publish-simple.yml instead of deleted publish-public.yml - Add validation for workflow_dispatch and npm_token input features - Maintains compatibility with existing publish-public.sh script - Fixes test suite failure for NPM registry publication requirements 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
1 parent f014512 commit f834e81

File tree

1 file changed

+4
-2
lines changed

1 file changed

+4
-2
lines changed

claude-dev-toolkit/tests/test_req_053_public_npm_publication.js

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -50,12 +50,14 @@ class PublicNpmPublicationTests {
5050
}
5151

5252
test_automated_public_workflow_exists() {
53-
const workflowPath = path.join(__dirname, '../../.github/workflows/publish-public.yml');
54-
assert(fs.existsSync(workflowPath), 'GitHub Actions workflow for public publishing must exist');
53+
const workflowPath = path.join(__dirname, '../../.github/workflows/npm-publish-simple.yml');
54+
assert(fs.existsSync(workflowPath), 'GitHub Actions workflow for NPM publishing must exist');
5555

5656
const content = fs.readFileSync(workflowPath, 'utf8');
5757
assert(content.includes('npm publish'), 'Workflow must include npm publish step');
5858
assert(content.includes('registry.npmjs.org'), 'Workflow must target public npm registry');
59+
assert(content.includes('workflow_dispatch'), 'Workflow must be manually triggerable');
60+
assert(content.includes('npm_token'), 'Workflow must have NPM token input');
5961
}
6062

6163
test_version_tagging_automation() {

0 commit comments

Comments
 (0)