Skip to content

Comments

fix: robust GitHub Actions publishing workflow#4

Merged
codedbytahir merged 1 commit intomainfrom
feature/advanced-export-ai-guidelines-8613210377134183394
Feb 14, 2026
Merged

fix: robust GitHub Actions publishing workflow#4
codedbytahir merged 1 commit intomainfrom
feature/advanced-export-ai-guidelines-8613210377134183394

Conversation

@codedbytahir
Copy link
Owner

@codedbytahir codedbytahir commented Feb 14, 2026

  • Fixed publish.yml to handle initial package publications (first-time publishing).
  • Added version-check protection to avoid redundant publish attempts.
  • Added NPM_TOKEN verification step for better error debugging.
  • Ensured all packages have publishConfig for public NPM registry.
  • Standardized package naming and synchronization for CI/CD.

Summary by CodeRabbit

  • Documentation
    • Restructured onboarding guide with CLI-first approach for easier project setup.
    • Added contributor setup instructions including development commands.
    • Updated Video Export feature description for clarity.

- Fixed `publish.yml` to handle initial package publications (first-time publishing).
- Added version-check protection to avoid redundant publish attempts.
- Added `NPM_TOKEN` verification step for better error debugging.
- Ensured all packages have `publishConfig` for public NPM registry.
- Standardized package naming and synchronization for CI/CD.

Co-authored-by: codedbytahir <200578194+codedbytahir@users.noreply.github.com>
@coderabbitai
Copy link

coderabbitai bot commented Feb 14, 2026

Caution

Review failed

The pull request is closed.

📝 Walkthrough

Walkthrough

The pull request updates the project's release automation and documentation practices. The CI workflow comment is clarified, the publish workflow gains NPM token verification and conditional publishing logic to prevent redundant releases, the main README is rewritten with CLI-first onboarding guidance, and the create-motionforge package adds explicit publish configuration.

Changes

Cohort / File(s) Summary
CI/CD Workflows
.github/workflows/ci.yml, .github/workflows/publish.yml
CI workflow comment updated for clarity; publish workflow enhanced with NPM token verification step and conditional publishing logic that checks local vs. published versions before deployment.
Package Configuration
packages/create-motionforge/package.json
Added publishConfig block specifying public access and npm registry URL for explicit publish settings.
Documentation
README.md, packages/motionforge/README.md
Main README restructured from dual-path onboarding to CLI-first narrative with guided setup steps; motionforge README feature description updated from WebCodecs/frame rendering to WebM encoding via MediaRecorder API.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Poem

🐰 Hop, skip, and CLI bound,
Workflows now check before they sound,
Publish once, not twice in haste,
Docs rewritten, no taste of waste,
Building castles, clean and grand!

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch feature/advanced-export-ai-guidelines-8613210377134183394

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@codedbytahir codedbytahir merged commit 7b7ecc2 into main Feb 14, 2026
1 of 2 checks passed
@coderabbitai coderabbitai bot mentioned this pull request Feb 14, 2026
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