Skip to content

refactor: extract Goal prompt and report runtimes - #134

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

refactor: extract Goal prompt and report runtimes#134
ByBrawe merged 4 commits into
mainfrom
refactor/goal-presentation-runtime

Conversation

@ByBrawe

@ByBrawe ByBrawe commented Aug 21, 2026

Copy link
Copy Markdown
Owner

Summary

Continue the Loop source modularization by splitting Goal presentation responsibilities out of runtime/goal-runtime.js.

  • add runtime/goal-prompt.js for persistent Goal prompt construction and file/context rendering;
  • add runtime/goal-report.js for report path/text/write behavior;
  • keep goal-runtime.js focused on Goal selection, evidence/progress helpers, and lifecycle mutations;
  • preserve the existing public source surface by re-exporting buildGoalPrompt, goalReportPath, goalReportText, and writeGoalReport from goal-runtime.js, so current consumers/tests do not need a compatibility migration;
  • syntax-gate the new source modules while retaining the existing Goal runtime regression suite.

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

Verification

Exact head 2584da1a10e15df67b5690c08b7c817905b14ad4:

  • 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:26
Copilot AI lite review requested due to automatic review settings August 21, 2026 06:26

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 494a1be 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