Skip to content

Release/3.13.0#661

Merged
bao-byterover merged 3 commits into
mainfrom
release/3.13.0
May 14, 2026
Merged

Release/3.13.0#661
bao-byterover merged 3 commits into
mainfrom
release/3.13.0

Conversation

@bao-byterover
Copy link
Copy Markdown
Collaborator

No description provided.

@bao-byterover bao-byterover requested a review from leehpham as a code owner May 14, 2026 13:26
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 14, 2026

Claude finished @bao-byterover's task in 2m 34s —— View job


Claude PR Review — Release/3.13.0

  • Read CHANGELOG.md changes
  • Read CLAUDE.md changes
  • Read package.json changes
  • Verify version consistency and referenced paths
  • Post inline + summary review

Overall: 4.5 / 5 — clean release PR

This is a small, mechanical release PR (3 docs/version files, +26/-1). I verified the CHANGELOG entries against the commits in origin/main..HEAD and confirmed all paths referenced in the new CLAUDE.md sections resolve in the tree. Two minor findings posted inline; neither is a release blocker.

praise: The CHANGELOG entries are well-written and consistently user-facing — each bullet leads with the user-visible effect, then the mechanism. The new "LLM Billing" section in CLAUDE.md is dense but accurate, and the connector-subsystem note pointing future agents at the footer-marker invariant in shared/constants.ts is exactly the sort of non-obvious context worth pinning.

Findings

# File Type Summary
1 package.json:15 suggestion (non-blocking) package-lock.json still says 3.11.0 at root and packages[""]. Pre-existing on main, but a release PR is the natural place to fix. npm install regenerates just those two fields.
2 CLAUDE.md:121 nitpick (non-blocking) provider-flow-dialog.tsx lives under components/provider-flow/, not directly under components/. Code suggestion attached.

Verification done

Comment thread package.json
"knowledge-management"
],
"version": "3.12.0",
"version": "3.13.0",
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

suggestion (non-blocking): package.json is now 3.13.0, but package-lock.json still has "version": "3.11.0" at both the root (line 3) and the empty-key packages[""].version (line 9). This is pre-existing on main (3.12.0 didn't bump it either) — npm ci won't fail on it, but the published lockfile will keep claiming a stale version. A one-liner npm install (no args) regenerates the lockfile and updates only those two version fields. Worth fixing here since this is the release PR.

Fix this →

Comment thread CLAUDE.md
- CLI status line: `brv status` / `brv providers list` render credits via `oclif/lib/billing-line.ts` + `format-billing-line.ts`
- `brv providers connect` (byterover provider) runs a team-select step that emits `BillingEvents.SET_PINNED_TEAM`
- WebUI: credits pill (`webui/features/provider/components/credits-pill.tsx`), team-select step in `provider-flow-dialog.tsx`, billing API wrappers in `webui/features/provider/api/` (`list-billing-usage`, `list-teams`, `get-pinned-team`, `set-pinned-team`, `get-free-user-limit`)

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nitpick (non-blocking): Path is slightly off — provider-flow-dialog.tsx actually lives at webui/features/provider/components/provider-flow/provider-flow-dialog.tsx (nested under a provider-flow/ subdirectory), not directly under components/. Same line also references credits-pill.tsx with the correct path, so this is just the dialog reference.

Suggested change
- WebUI: credits pill (`webui/features/provider/components/credits-pill.tsx`), team-select step in `provider-flow/provider-flow-dialog.tsx`, billing API wrappers in `webui/features/provider/api/` (`list-billing-usage`, `list-teams`, `get-pinned-team`, `set-pinned-team`, `get-free-user-limit`)

@bao-byterover bao-byterover merged commit 93f2514 into main May 14, 2026
11 checks passed
@bao-byterover bao-byterover deleted the release/3.13.0 branch May 18, 2026 03:31
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