Setup repository for npm publishing#5
Conversation
- Add MIT LICENSE for OpenStaticFish - Add .npmignore to exclude dev files - Rename package to @openstaticfish/actionflow v0.1.0 - Add npm publishing metadata (bin, files, exports, engines) - Update tsconfig.json for declaration generation - Update README.md with new package name and install instructions - Update Header.tsx UI title to ACTIONFLOW - Update AGENTS.md header reference Prepares repository for publishing to npm registry
Fixes the heredoc syntax issue that was causing workflow validation failures. Replaced problematic heredoc with jq-based JSON construction for proper escaping of special characters in PR diff and comments. Changes: - workflows/opencode/pr/opencode-pr.yml: Use jq to build API payload - workflows/opencode/pr/opencode-pr-nix.yml: Same fix for Nix variant - .github/workflows/opencode-pr.yml: Apply fix to installed workflow This resolves the "Invalid workflow file" error that prevented the OpenCode AI PR review from running.
The previous fix had issues with backtick escaping in jq strings. Replaced inline jq string construction with printf for building the prompt, then pass the complete prompt to jq using --arg. This avoids escape character issues with backticks and newlines in the JSON construction.
Replaced broken heredoc/curl approach with anomalyco/opencode/github@latest GitHub Action, matching the working implementation from ZigCraft. Templates updated: - workflows/opencode/pr/opencode-pr.yml - workflows/opencode/opencode/opencode.yml - workflows/opencode/triage/opencode-triage.yml This fixes the YAML syntax errors and API integration issues.
📋 SummaryNo linked issues found in the PR description. This PR prepares the repository for npm publishing as 🔴 Critical Issues (Must Fix - Blocks Merge)None identified.
|
| Principle | Score | Notes |
|---|---|---|
| Single Responsibility | 9 | Each file has a clear, focused purpose |
| Open/Closed | 8 | Configuration changes are additive; existing functionality unchanged |
| Liskov Substitution | 10 | No inheritance hierarchies modified |
| Interface Segregation | 10 | Clean separation between package config and code |
| Dependency Inversion | 10 | No coupling issues introduced |
| Average | 9.4 |
🎯 Final Assessment
Overall Confidence Score: 95%
Confidence Breakdown:
- Code Quality: 95% (clean, well-organized configuration changes)
- Completeness: 95% (all necessary npm publishing artifacts included)
- Risk Level: 5% (low risk - configuration-only changes)
- Test Coverage: N/A% (no test changes required for configuration PR)
Merge Readiness:
- All critical issues resolved
- SOLID average score >= 6.0
- Overall confidence >= 60%
- No security concerns
- Tests present and passing (if applicable)
Verdict:
MERGE
This is a clean, well-prepared npm publishing setup PR with no blocking issues.
1 participant
This PR prepares the repository for publishing to npm as
@openstaticfish/actionflow.Changes
New Files
LICENSE- MIT License for OpenStaticFish.npmignore- Excludes source files and dev dependencies from npm packageUpdated Files
package.json- Renamed to@openstaticfish/actionflowv0.1.0 with:actionflowandafprepublishOnlytsconfig.json- Added TypeScript declaration generation supportREADME.md- Updated with new package name and install instructionsAGENTS.md- Updated header referencesrc/tui/components/Header.tsx- Updated UI title to "ACTIONFLOW"Build Verification
react-devtools-core)dist/)npm pack --dry-runshows 50 files ready for publishNext Steps After Merge
npm loginTesting
This PR should trigger the OpenCode AI PR review workflow to verify the changes.