Skip to content

FE-698: On more context pack + spec for agent CLI v1#114

Open
lunelson wants to merge 1 commit into
ln/fe-698-slice-efrom
ln/fe-698-slice-f
Open

FE-698: On more context pack + spec for agent CLI v1#114
lunelson wants to merge 1 commit into
ln/fe-698-slice-efrom
ln/fe-698-slice-f

Conversation

@lunelson
Copy link
Copy Markdown
Contributor

@lunelson lunelson commented May 8, 2026

No description provided.

Copy link
Copy Markdown
Contributor Author

lunelson commented May 8, 2026

@lunelson lunelson force-pushed the ln/fe-698-slice-e branch from bbb63ab to 27de9e2 Compare May 8, 2026 14:11
@lunelson lunelson force-pushed the ln/fe-698-slice-f branch from 88907d8 to 1f22e49 Compare May 8, 2026 14:11
@lunelson lunelson marked this pull request as ready for review May 11, 2026 14:18
@cursor
Copy link
Copy Markdown

cursor Bot commented May 11, 2026

PR Summary

Low Risk
Low risk: adds a new proposal-only reconciliation scenario, prompt asset, and tests without changing existing runtime flows or mutating capabilities.

Overview
Adds a new proposal-only reconciliation prompt scenario that renders open reconciliation_need items with source/target intent anchors, status/kind, and optional reasons.

Extends the scenario/context-pack types to include reconciliation, wires it into the prompt loader via a new reconciliation-system.md asset, and updates the scenario runner to emit deterministic no-provider probe artifacts with read-only capabilities. Unit tests cover both context-pack rendering and scenario artifact construction, and planning docs are updated to record the completed slice.

Reviewed by Cursor Bugbot for commit 582e418. Bugbot is set up for automated code reviews on this repo. Configure here.

@augmentcode
Copy link
Copy Markdown

augmentcode Bot commented May 11, 2026

🤖 Augment PR Summary

Summary: Adds a proposal-only “reconciliation” prompt/context scenario to help plan how to address open reconciliation needs without mutating intent-graph truth.

Changes:

  • Introduced buildReconciliationContextPack / renderReconciliationContextPack to render objectives, open reconciliation_need entries (with source/target anchors), and an explicit proposal boundary.
  • Wired the new scenario through src/server/context-pack.ts, src/server/prompt-loader.ts (new reconciliation.system prompt), and src/server/scenario-runner.ts (new scenario definition + default read-only capabilities).
  • Added tests covering deterministic reconciliation context rendering and probe artifact generation.
  • Updated spec/plan docs to capture agent-native CLI design considerations and record the reconciliation context-pack slice completion.

Technical Notes: Reconciliation artifacts remain no-provider/proposal-only by default and expose only read-only capabilities (e.g., scenario.render, intentGraph.validateEdge).

🤖 Was this summary useful? React with 👍 or 👎

Copy link
Copy Markdown

@augmentcode augmentcode Bot left a comment

Choose a reason for hiding this comment

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

Review completed. 1 suggestion posted.

Fix All in Augment

Comment augment review to trigger a new review at any time.

Comment thread memory/SPEC.md Outdated
@lunelson lunelson force-pushed the ln/fe-698-slice-f branch from b45e2da to 76fabef Compare May 11, 2026 15:50
@lunelson lunelson changed the title add spec items re agent-native CLI design FE-698: On more context pack + spec for agent CLI v1 May 11, 2026
Copy link
Copy Markdown

@cursor cursor Bot left a comment

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 76fabef. Configure here.

Comment thread src/server/context-pack/reconciliation.ts Outdated
@lunelson lunelson force-pushed the ln/fe-698-slice-f branch from 76fabef to 582e418 Compare May 11, 2026 16:17
@lunelson lunelson force-pushed the ln/fe-698-slice-e branch from 99cc970 to 0fb5687 Compare May 11, 2026 16:17
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