Skip to content

🧪 Preserve API-owned region context - #331

Merged
Robdel12 merged 1 commit into
mainfrom
rd/compact-dynamic-region-context
Jul 26, 2026
Merged

🧪 Preserve API-owned region context#331
Robdel12 merged 1 commit into
mainfrom
rd/compact-dynamic-region-context

Conversation

@Robdel12

@Robdel12 Robdel12 commented Jul 26, 2026

Copy link
Copy Markdown
Contributor

Why

Vizzly now owns compact versus complete dynamic-region response detail in API PR #653. The CLI should preserve those server-provided facts without understanding exclusion reasons or duplicating API rules.

What changed

The real CLI boundary test now serves distinct compact and complete API responses, then verifies the CLI preserves each dynamic_regions shape exactly. Compact agent requests continue using details=summary or details=diffs; explicit --full continues requesting the complete response.

There is no dynamic-region filtering or exclusion taxonomy in CLI product code. Local TDD behavior and the rest of the agent evidence formatting remain unchanged.

Confidence

The real CLI process passes all compact, full, pagination, diff-detail, source-provenance, comparison, and local-workspace context scenarios. This follow-on is intentionally test-only because the existing client pass-through was already the correct implementation once the API owned response detail.

Depends on the merged Vizzly API PR #653.

Exercise compact and complete dynamic-region responses through the real CLI while keeping API exclusion semantics out of client code.
@Robdel12
Robdel12 force-pushed the rd/compact-dynamic-region-context branch from 441571a to c8534d1 Compare July 26, 2026 17:34
@Robdel12 Robdel12 changed the title 🐛 Keep agent region context actionable 🧪 Preserve API-owned region context Jul 26, 2026
@Robdel12
Robdel12 merged commit 4797835 into main Jul 26, 2026
29 of 30 checks passed
@Robdel12
Robdel12 deleted the rd/compact-dynamic-region-context branch July 26, 2026 17:39
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