Skip to content

feat(harness): /orchestrator:feedback skill — file PO feedback from consumer repos (issue #177) - #199

Merged
robercano-ghbot merged 1 commit into
mainfrom
feat/issue-177-feedback-skill
Jul 29, 2026
Merged

feat(harness): /orchestrator:feedback skill — file PO feedback from consumer repos (issue #177)#199
robercano-ghbot merged 1 commit into
mainfrom
feat/issue-177-feedback-skill

Conversation

@robercano-ghbot

Copy link
Copy Markdown
Collaborator

Closes #177

What

Ships the /orchestrator:feedback skill: a five-second capture path that files owner PO-feedback into robercano/reCode from a consumer repo (reDeploy/reDeFi), with autofilled metadata.

How

  • .claude/skills/feedback/SKILL.md (name: feedback → exposes /orchestrator:feedback, same auto-registration as the setup/sync skills). Prompts only for the description + optional severity.
  • .claude/skills/feedback/feedback.sh — deterministic helper: derives the from:redeploy/from:redefi label from the origin remote, reads the installed plugin version from the cache manifest (${CLAUDE_PLUGIN_ROOT}/.claude-plugin/plugin.json, with local fallback → unknown), assembles an observed/expected/severity body template, applies the feedback label, and files into robercano/reCode with no milestone / no planned (triage inbox). Guards against non-consumer contexts (reCode itself, no origin, plugin not installed) and never creates a partial issue.
  • Uses plain owner gh by design (documented) — it IS owner feedback filing an issue (never a PR), the one intentional exception to the bot-gh rule.
  • .claude/scripts/feedback.test.sh — 37 hermetic, offline checks (dry-run + overridable $FEEDBACK_GH_BIN stub; auto-discovered by the test gate).
  • docs/USAGE.md — new subsection in the Release-cycle area.
  • No hooks touched → Self-gate: assert hooks parity between .claude/settings.json and hooks/hooks.json #140 hooks-parity check stays green.

Gates (via GATES_FILE=.claude/self/gates.json)

  • build: PASS (JSON valid, hooks parity OK)
  • lint: PASS (shellcheck + workflow syntax)
  • test: PASS (checks.sh test incl. feedback.test.sh 37/37, + smoke-fanout.sh)

Review

  • correctness (opus): APPROVE
  • tests (sonnet): APPROVE
  • Non-blocking nits only (file mode 100644 vs 100755; cosmetic trailing-slash on origin URL) — deliberately left out of scope to keep the change minimal.

🤖 Generated with Claude Code

…sue #177)

Ship a five-second feedback-capture skill for rollout testing: run
/orchestrator:feedback "<one-line description>" from a consumer repo
(reDeploy/reDeFi) and it autofills the origin label, installed plugin
version, and body template, then files a triage-inbox issue (no
milestone, no planned) in robercano/reCode via plain gh — the one
deliberate owner-identity gh call in this plugin, since it IS the
owner's own feedback.

Co-Authored-By: Claude Sonnet 5 <[email protected]>
@robercano-ghbot robercano-ghbot added the needs-human Loop escalated: attempt budget exhausted, manual triage required label Jul 29, 2026
@robercano-ghbot

Copy link
Copy Markdown
Collaborator Author

feat(harness): /orchestrator:feedback skill — file PO feedback from consumer repos (issue #177) (not yet reviewed)

@robercano-ghbot robercano-ghbot removed the needs-human Loop escalated: attempt budget exhausted, manual triage required label Jul 29, 2026
@robercano-ghbot
robercano-ghbot merged commit 9e85703 into main Jul 29, 2026
9 checks passed
@robercano-ghbot
robercano-ghbot deleted the feat/issue-177-feedback-skill branch July 29, 2026 11:08
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.

/orchestrator:feedback skill — file PO feedback from consumer repos with autofilled metadata

2 participants