Skip to content

feat(review-tutor): publish public package - #85

Merged
ElbertePlinio merged 2 commits into
mainfrom
feat/review-tutor-public-package
Aug 26, 2026
Merged

feat(review-tutor): publish public package#85
ElbertePlinio merged 2 commits into
mainfrom
feat/review-tutor-public-package

Conversation

@ElbertePlinio

Copy link
Copy Markdown
Member

Changed

  • publish @pickforge/review-tutor at the workspace lockstep version
  • add public npm metadata and Node.js engine requirement
  • keep the installed CLI at dist/bin.js and make the Pi peer optional
  • include current resources plus the Claude Code and Codex directories landing in PR C

Tested

  • npm pack --dry-run: includes dist/bin.js, extensions/review-tutor.ts, src/**/*.ts, skills/review-tutor/SKILL.md, README.md, and package.json; excludes test/, node_modules, and .tsbuildinfo
  • installed the packed tarball in a temporary npm project; review-tutor --help printed usage and exited 0
  • ran review-tutor worktree --no-open in a temporary Git repo; authenticated /api/state returned 200 with harnesses, then SIGTERM exited 0
  • installed the temporary package path with Pi using an isolated PI_CODING_AGENT_DIR
  • bun run check

Not tested

  • real npm publish; the next release tag performs it

Risk class: standard

Refs #82

@ElbertePlinio
ElbertePlinio force-pushed the feat/review-tutor-public-package branch from 45e6f6f to 203c3d4 Compare August 26, 2026 15:16
@ElbertePlinio

Copy link
Copy Markdown
Member Author

Review (Opus, standard): package.json correct and release-safe; git+ repository url applied (46b16ab). README P1 is resolved by stacking: #84 (README install rewrite + adapters) now bases on this PR and merges before any tag, so nothing publishes with the stale README. Note for release: do not run publish.yml via workflow_dispatch before the 0.13.0 bump — the eight other packages are already at 0.12.0 and would conflict while this one would burn 0.12.0 outside a tag.

@ElbertePlinio ElbertePlinio reopened this Aug 26, 2026
@ElbertePlinio
ElbertePlinio force-pushed the feat/review-tutor-public-package branch 2 times, most recently from 50bbfc5 to ad434c0 Compare August 26, 2026 19:47
@ElbertePlinio

Copy link
Copy Markdown
Member Author

Ready for human review. Rebased onto #83 at 40cbc82; CI green at ad434c0 (run 33007071022). No source changes since the review-clean head.

@ElbertePlinio
ElbertePlinio force-pushed the feat/review-tutor-public-package branch from ad434c0 to 273de2c Compare August 26, 2026 21:21
@ElbertePlinio
ElbertePlinio merged commit 5e7a8a8 into main Aug 26, 2026
4 checks passed
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