Skip to content

feat(overengineering): roll up out-of-repo findings per owner (#4596) - #4835

Merged
cursor[bot] merged 4 commits into
mainfrom
cursor/4596-overengineering-owner-rollup-37e9
Sep 28, 2026
Merged

cursor[bot] merged 4 commits into
mainfrom
cursor/4596-overengineering-owner-rollup-37e9

Conversation

@kyle-sexton

Copy link
Copy Markdown
Contributor

Closes #4596

Summary

/overengineering:audit still walks every settings scope. The inline report lists in-repo findings first and in full, then one summary row per out-of-repo owner. Finding ids and artifact rows are unchanged. A custody filter that would skip those layers is declined.

Fix

  • Report-template "Custody grouping": in-repo first; out-of-repo collapses to count, verdict mix, and a pointer at the artifact.
  • SKILL.md Arguments names the report view and stamps that the walk is unchanged.
  • Eval 13 covers the rollup and forbids a custody=repo argument.

Decision

Verification

  • Eval JSON schema validates; check-evals-quality PASS.
  • overengineering 0.4.15 changelog + manifest.
Open in Web Open in Cursor 

cursoragent and others added 3 commits September 28, 2026 03:15
…dit report (#4596)

A direct audit still walks every settings scope. The inline report lists
in-repo findings first and in full, then one summary row per out-of-repo
owner. Finding ids and artifact rows are unchanged. A custody filter is
declined: hiding those layers would skip incumbents the skill still has
to name.

Co-authored-by: Kyle Sexton <kyle-sexton@users.noreply.github.com>
The inline report lists in-repo findings first, then one summary row per
out-of-repo owner. Artifact rows and finding ids stay unchanged.

Co-authored-by: Kyle Sexton <kyle-sexton@users.noreply.github.com>
…eering-owner-rollup-37e9

Co-authored-by: Kyle Sexton <kyle-sexton@users.noreply.github.com>
@github-actions

Copy link
Copy Markdown
Contributor

PR body contract — issue linkage

This PR body does not yet satisfy the issue-linkage contract:

  • Missing a "## Related" section. List related PRs, ADRs, or decision-log entries this PR does not close.

Edit the body and this comment updates itself on the next run.

@cursor
cursor Bot marked this pull request as ready for review September 28, 2026 03:30
…eering-owner-rollup-37e9

Co-authored-by: Kyle Sexton <kyle-sexton@users.noreply.github.com>
@cursor
cursor Bot merged commit 5f25337 into main Sep 28, 2026
19 checks passed
@cursor
cursor Bot deleted the cursor/4596-overengineering-owner-rollup-37e9 branch September 28, 2026 03:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

overengineering:audit: out-of-repo manifests bury repo findings in a single-repo audit; add a custody filter or a per-owner rollup

2 participants