feat(activity): group activity by branch#978
Open
prateek wants to merge 1 commit into
Open
Conversation
roborev: Combined Review (
|
prateek
changed the title
prateek
force-pushed
the
split-branch-activity-rollups
branch
from
8e778e2 to
10b6e57
Compare
Contributor
Author
|
Addressed the automated review finding: the |
roborev: Combined Review (
|
Adds branch as a rollup dimension to activity reports: by_branch rows alongside by_project/by_model/by_agent, aggregated identically in SQLite, Postgres, and DuckDB with parity tests. Rows carry typed project and branch fields, so same-named branches in different projects stay distinct and no internal token encoding leaks into the JSON. Surfaces: a "By branch" section in the CLI activity report, and a Branch typeahead plus Branch breakdown panel on the web Activity page. The activity store fetches its own branch list from the /branches endpoint, so the page filter works standalone.
prateek
force-pushed
the
split-branch-activity-rollups
branch
from
10b6e57 to
d6707d2
Compare
roborev: Combined Review (
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Activity reports now include an activity-specific branch breakdown, filling the grouping piece that the shared branch filter foundation left to each view. The new
by_branchrows are keyed by typedprojectandbranchfields, so same-named branches in different projects stay separate and the API does not expose the internal filter token as a grouping key.The CLI report prints a
By branchsection. The web Activity page adds a branch typeahead and a Branch breakdown panel, with the activity store loading branch options itself so the page can deep-link and refresh without depending on another view.SQLite, PostgreSQL, and DuckDB all carry the same
by_branchshape. Empty branches remain distinct from a literal branch namedunknown; reviewers should look at the activity aggregation path, the three read backends, and the Activity page breakdown wiring.Part of #928