Skip to content

Fix: preserve independent intent boundaries - #243

Merged
ivory-code merged 1 commit into
mainfrom
fix/independent-intent-boundaries
Sep 2, 2026
Merged

Fix: preserve independent intent boundaries#243
ivory-code merged 1 commit into
mainfrom
fix/independent-intent-boundaries

Conversation

@ivory-code

Copy link
Copy Markdown
Contributor

Summary

  • Partition commitless mixed diffs by anchored repository evidence instead of collapsing all residual files into one intent.
  • Keep distinct commits that edit one shared source file separate, including exact literal navigation evidence.
  • Preserve separate intent-backed QA flows through downstream flow deduplication.

Behavioral Contract

  • Unrelated changed files remain separate unless a path, changed symbol, or explicit flow annotation anchors their relationship.
  • Common imports, accessibility attributes, UI controls, and broad routing verbs cannot join independent changes.
  • Separate intents that edit one route registry retain their own destination evidence and final QA flow.
  • Related multi-file behavior and cleanup-tip ranking continue to work as before.

Evidence

Closes #242.

Positive and negative coverage:

  • web-independent-diff-intents
  • web-independent-route-commits
  • release-shaped web changes recover diff-first sharing, access, time, media, and storage QA
  • web-preferences-cleanup-tip
  • long-pr-independent-intents

Checks

Check only what applies. Mark the rest N/A in Review Notes.

  • Focused regression test
  • pnpm test
  • pnpm bench:ci for inference, routing, trace, or output
  • pnpm bench:execution for E2E compiler or execution fixtures
  • pnpm scan for scanner, security, or repository policy
  • pnpm plugin:check and pnpm plugin:smoke for plugin changes
  • Documentation links and commands verified

Public OSS Check

  • No private repository, source, path, customer data, credential, or internal smoke output is included.
  • Shared inference has unrelated positive and negative coverage, or this is not applicable.
  • User-facing commands and claims match actual behavior.

Review Notes

pnpm check and git diff --check also pass. pnpm bench:execution, pnpm scan, and plugin checks are N/A because this change does not modify execution compilation, scanner policy, security policy, or plugin packaging. A static QAMap comparison selected the focused change-intent suite and pnpm bench:ci; it remained correctly marked not-run, and both commands were executed independently.

@ivory-code ivory-code added type: fix Bug fix or corrective follow-up area: qa-planning PR-aware QA planning, coverage, evidence, and fixture readiness labels Sep 2, 2026
@ivory-code ivory-code self-assigned this Sep 2, 2026
@ivory-code
ivory-code merged commit c27c61e into main Sep 2, 2026
2 checks passed
@ivory-code
ivory-code deleted the fix/independent-intent-boundaries branch September 2, 2026 05:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area: qa-planning PR-aware QA planning, coverage, evidence, and fixture readiness type: fix Bug fix or corrective follow-up

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Fix: preserve independent intent boundaries within mixed diffs

1 participant