Skip to content

feat(review-tutor): add harness adapters - #84

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

feat(review-tutor): add harness adapters#84
ElbertePlinio merged 2 commits into
mainfrom
feat/review-tutor-harness-adapters

Conversation

@ElbertePlinio

@ElbertePlinio ElbertePlinio commented Aug 26, 2026

Copy link
Copy Markdown
Member

Changed

  • add the Claude Code Review Tutor plugin and Pickforge marketplace entry
  • add the Codex Review Tutor skill
  • document Pi, Claude Code, Codex, and shell installation, including the pre-v0.13.0 checkout path
  • make the Claude skill pass one validated, single-quoted source argument through Bash and never fetch the returned URL
  • add marketplace metadata and set the Claude plugin manifest to v0.12.0

Release note: the Claude plugin manifest version and the npm package version must move together; PR B sets both to v0.12.0.

Tested

  • bun run check
  • claude plugin validate --strict .claude-plugin/marketplace.json
  • claude plugin validate --strict packages/review-tutor/claude-plugin
  • isolated temporary Claude local-path marketplace install, list, details, uninstall, and post-uninstall empty list
  • isolated Codex skill copy and structural validation

Not tested

  • invoking /review-tutor through Claude Code or Codex, because request-spending commands were explicitly excluded
  • remote git-subdir installation before the plugin exists on the marketplace's unpinned main source

Risk class: standard

Refs #82

@ElbertePlinio
ElbertePlinio force-pushed the feat/review-tutor-harness-adapters branch from 8fc83f2 to 98ae289 Compare August 26, 2026 15:13
@ElbertePlinio

Copy link
Copy Markdown
Member Author

Ready for human review at 98ae289 (base #83). Review: P1 shell-injection path in the skill's ! block removed (agent-composed Bash call, single-quoted argv, charset allowlist), README caveat until v0.13.0, marketplace description, plugin version 0.12.0 in lockstep. claude plugin validate --strict passes for plugin and marketplace; local marketplace install/uninstall proof done. Follow-up hardening (CLI-side charset check) lands on #83.

@ElbertePlinio
ElbertePlinio force-pushed the feat/review-tutor-harness-adapters branch from 98ae289 to 5be3066 Compare August 26, 2026 15:15
@ElbertePlinio
ElbertePlinio force-pushed the feat/review-tutor-harness-adapters branch from 5be3066 to da2af43 Compare August 26, 2026 15:19
@ElbertePlinio
ElbertePlinio changed the base branch from feat/review-tutor-cli to feat/review-tutor-public-package August 26, 2026 15:19
@ElbertePlinio ElbertePlinio reopened this Aug 26, 2026
@ElbertePlinio
ElbertePlinio force-pushed the feat/review-tutor-public-package branch from 46b16ab to 50bbfc5 Compare August 26, 2026 19:44
@ElbertePlinio
ElbertePlinio force-pushed the feat/review-tutor-harness-adapters branch from da2af43 to 8a1d08c Compare August 26, 2026 19:45
@ElbertePlinio
ElbertePlinio force-pushed the feat/review-tutor-public-package branch from 50bbfc5 to ad434c0 Compare August 26, 2026 19:47
@ElbertePlinio
ElbertePlinio force-pushed the feat/review-tutor-harness-adapters branch from 8a1d08c to 1ab82f9 Compare August 26, 2026 19:47
@ElbertePlinio

Copy link
Copy Markdown
Member Author

Rebased onto #85 at ad434c0; CI green at 1ab82f9 (run 33007073567). No source changes since the review-clean head.

@ElbertePlinio
ElbertePlinio changed the base branch from feat/review-tutor-public-package to main August 26, 2026 21:20
@ElbertePlinio
ElbertePlinio force-pushed the feat/review-tutor-harness-adapters branch from 1ab82f9 to 0987113 Compare August 26, 2026 21:21
@ElbertePlinio
ElbertePlinio changed the base branch from main to feat/review-tutor-public-package August 26, 2026 21:21
@ElbertePlinio
ElbertePlinio changed the base branch from feat/review-tutor-public-package to main August 26, 2026 21:23
@ElbertePlinio
ElbertePlinio force-pushed the feat/review-tutor-harness-adapters branch from 0987113 to c920c30 Compare August 26, 2026 21:23
@ElbertePlinio
ElbertePlinio merged commit e6a6345 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