Skip to content

Add cluster discovery, review links, and focused diffs - #5

Merged
CalLavicka merged 6 commits into
mainfrom
2026-09-25/cluster-commands
Sep 28, 2026
Merged

CalLavicka merged 6 commits into
mainfrom
2026-09-25/cluster-commands

Conversation

@CalLavicka

@CalLavicka CalLavicka commented Sep 25, 2026 •

Copy link
Copy Markdown
Member

Adds clusters [pr-url] and cluster <id> so agents can discover review areas, share version-pinned Assert links, and inspect selected changes. Repository and PR default to the current checkout/branch, with --repo and --pr overrides.

Text lists show titles, explicit IDs, descriptions, file counts, and review links. The default view shows one level of subclusters; --depth all expands the full tree without repeating file paths. Cluster details include parent/subcluster navigation, accurate additions/deletions, and the five largest changed files, with --files to show every file. --diff writes only the selected diff to stdout; errors and timeout diagnostics go to stderr. JSON preserves the full structured results.

Commands support bounded waiting, partial results, --wait-until ready, and --no-wait, with distinct error/timeout exit codes. Diffs use the returned base/head comparison and include descendant ranges, including changes sharing a hunk, renames, and binary metadata.

Validation: typecheck, formatting, build, and all 59 tests pass. Production checks covered large nested reviews, full-tree expansion, file-list limits, JSON output, raw diff stdout, and error/timeout stderr. Independently checked line statistics using Git. Fixtures use generic repositories and synthetic changes.

@assert-app

assert-app Bot commented Sep 25, 2026 •

Copy link
Copy Markdown

Review on Assert →

2 clusters identified

Merge candidate is preparing...

@CalLavicka
CalLavicka marked this pull request as ready for review September 25, 2026 23:13
@CalLavicka
CalLavicka merged commit 063f298 into main Sep 28, 2026
3 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