Refactor Holon solve/workflow to explicit goal routing

[jolestar]

Summary

This PR switches Holon solve/workflow routing to explicit goal-driven behavior and removes goal-hint/skill-based solve routing:

• Add holon solve --goal and make explicit goal highest priority.
• Remove solve-time workflow.json trigger.goal_hint dependency and skill-derived goal assembly.
• Add default PR intent routing in holon solve:
  • fix when unresolved review threads exist, or latest review state is CHANGES_REQUESTED
  • otherwise review
• Add reusable workflow inputs.goal and remove inputs.skill.
• Replace gate output goal_hint with command_text and add Resolve execution goal step.
• Pass resolved goal into action (with.goal) and action forwards --goal to holon solve.
• Update trigger examples/templates with optional goal override comments.

Validation

• go test ./cmd/holon
• YAML parse checks for:
  • .github/workflows/holon-solve.yml
  • .github/workflows/holon-trigger.yml
  • examples/workflows/holon-trigger.yml
  • cmd/holon/templates/workflow.yml
  • action.yml

[vercel]

The latest updates on your projects. Learn more about Vercel for GitHub.

Project	Deployment	Actions	Updated (UTC)
holon	Ready	Preview, Comment	Feb 27, 2026 6:15pm

[cloudflare-workers-and-pages]

Deploying holon with    Cloudflare Pages

Latest commit:	ecfd673
Status:	✅  Deploy successful!
Preview URL:	https://b8ac8679.holon-1dl.pages.dev
Branch Preview URL:	https://refactor-goal-driven-holon-s.holon-1dl.pages.dev

View logs

[Copilot]

Pull request overview

This PR refactors Holon’s solve/workflow routing to be explicitly goal-driven, adding a --goal flag and updating the GitHub workflows/action to resolve and pass an execution goal instead of relying on goal-hints/skill-based routing.

Changes:

• Add holon solve --goal and implement default goal generation for PRs (review vs fix) and issues.
• Update the composite action and reusable workflows to accept/resolve goal and forward it into holon solve.
• Update templates/examples and tests to reflect the new goal-driven routing.

Reviewed changes

Copilot reviewed 7 out of 7 changed files in this pull request and generated 3 comments.

Show a summary per file

File	Description
cmd/holon/solve.go	Adds explicit goal flag and new default goal generation (incl. PR review-signal inspection).
cmd/holon/solve_test.go	Updates goal-building tests to cover new PR/issue goal text helpers.
.github/workflows/holon-solve.yml	Replaces goal_hint with command_text, adds “Resolve execution goal”, and passes resolved goal to the action.
.github/workflows/holon-trigger.yml	Adds template comments for optional goal override.
action.yml	Replaces deprecated skill input with goal and forwards it to holon solve --goal.
cmd/holon/templates/workflow.yml	Updates workflow template comments for optional goal override.
examples/workflows/holon-trigger.yml	Updates example workflow comments for optional goal override.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[jolestar]

Addressed all 3 review comments in commit ecfd673:

1. Workflow regex word-boundary bug

• Replaced \b with POSIX-safe boundary matching in .github/workflows/holon-solve.yml.

2. Issue goal contract consistency

• Updated issue-side workflow-resolved goal text to include the same manifest/status/outcome contract language used by CLI defaults.

3. PR intent routing test coverage

• Added inferPRIntentFromSignals(...) helper in cmd/holon/solve.go.
• Added table-driven tests in cmd/holon/solve_test.go covering:
  • unresolved threads => fix
  • CHANGES_REQUESTED => fix
  • no signals => review

Validation run:

• go test ./cmd/holon
• YAML parse check for .github/workflows/holon-solve.yml

[holonbot]

Verified that all 3 review comments from copilot-pull-request-reviewer have been addressed in commit ecfd673:

1. Workflow regex word-boundary bug (.github/workflows/holon-solve.yml:709)

   • Fixed with POSIX-safe boundary pattern: (^|[^[:alnum:]_])(review|re-review|recheck)($|[^[:alnum:]_])

2. Issue goal contract consistency (.github/workflows/holon-solve.yml:716)

   • Updated issue goal text to include full manifest/status/outcome contract language matching buildIssueGoal()

3. PR intent routing test coverage (cmd/holon/solve.go)

   • Added inferPRIntentFromSignals() helper
   • Added table-driven tests in cmd/holon/solve_test.go covering:
     • unresolved threads => fix
     • CHANGES_REQUESTED => fix
     • no signals => review

All review threads have been resolved. Changes verified via go test ./cmd/holon - all new tests pass.