Skip to content

refactor: extract Goal evidence helpers - #135

Merged
ByBrawe merged 4 commits into
mainfrom
refactor/goal-evidence-runtime
Aug 21, 2026
Merged

refactor: extract Goal evidence helpers#135
ByBrawe merged 4 commits into
mainfrom
refactor/goal-evidence-runtime

Conversation

@ByBrawe

@ByBrawe ByBrawe commented Aug 21, 2026

Copy link
Copy Markdown
Owner

Summary

Continue the Goal modularization by extracting pure evidence/check/progress decision helpers from runtime/goal-runtime.js.

  • add runtime/goal-evidence.js for concrete-evidence classification, configured-check status, completion-check requirements, progress snapshots, and meaningful-progress detection;
  • keep goal-runtime.js focused on Goal selection/tool-argument normalization and lifecycle persistence mutations;
  • preserve the existing source API by re-exporting all extracted helpers from goal-runtime.js;
  • make goal-policy.js depend directly on the dedicated evidence module instead of the lifecycle facade;
  • syntax-gate the new module while keeping the existing Goal runtime/policy regressions as compatibility coverage.

Behavior-preserving source modularization only. No runtime state schema or user-visible command behavior change. The committed single-file bundle is intentionally unchanged in this source-only slice; the Bundle Gate regenerated and exercised the exact source head on Linux and Windows.

Verification

Exact head a8256b7f58057b54483f3b6335c0bbaf620bf60f:

  • CI: success on Linux/Windows, OpenCode 1.4.0/latest compatibility, and real OpenCode Loop hosts;
  • Single-file Bundle Gate: success on Linux/Windows, including source regeneration, generated-entry import, real-host Loop/queued-Goal canaries, and generated-bundle regression suite;
  • V2 Lifecycle Contract: success.

@ByBrawe
ByBrawe marked this pull request as ready for review August 21, 2026 06:32
Copilot AI lite review requested due to automatic review settings August 21, 2026 06:32

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@ByBrawe
ByBrawe merged commit ecef45c into main Aug 21, 2026
10 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.

2 participants