Uh oh!
There was an error while loading. Please reload this page.
Fetch upstream - #150
Conversation
Co-authored-by: LukeParkerDev <10430890+Hona@users.noreply.github.com>
Co-authored-by: Aiden Cline <rekram1-node@users.noreply.github.com>
@fwang is attempting to deploy a commit to the adevloper152's projects Team on Vercel. A member of the Team first needs to authorize it. |
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Repository UI Review profile: CHILL Plan: Pro Plus Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (29)
📝 WalkthroughWalkthroughChangesThe PR updates referral reward history, Azure GPT reasoning selection, session diff selection and comment behavior, diff styling, theme colors, and package versions. Referral rewards
Azure reasoning options
Diff interactions and styling
Package release versions
Estimated code review effort: 3 (Moderate) | ~25 minutes Possibly related PRs
Suggested labels: Suggested reviewers: 🚥 Pre-merge checks | ✅ 3 | ❌ 2❌ Failed checks (2 warnings)
✅ Passed checks (3 passed)
✨ Finishing Touches 💡 2⚔️ Resolve merge conflicts 💡
🛠️ Fix failing CI checks 💡
🧪 Generate unit tests (beta)
Comment |
 |
| Overall Grade Focus Area: Reliability | Security Reliability Complexity Hygiene |
Code Review Summary
| Analyzer | Status | Updated (UTC) | Details |
|---|---|---|---|
| JavaScript |  | Aug 4, 2026 12:55a.m. | Review ↗ |
| Shell |  | Aug 4, 2026 12:55a.m. | Review ↗ |
| Secrets | | Aug 4, 2026 12:55a.m. | Review ↗ |
| Docker | | Aug 4, 2026 12:55a.m. | Review ↗ |
| Python | | Aug 4, 2026 12:55a.m. | Review ↗ |
| CSS |  | Aug 4, 2026 12:55a.m. | Review ↗ |
| Rust | | Aug 4, 2026 12:55a.m. | Review ↗ |
| Ruby | | Aug 4, 2026 12:55a.m. | Review ↗ |
| Swift | | Aug 4, 2026 12:55a.m. | Review ↗ |
| PHP | | Aug 4, 2026 12:55a.m. | Review ↗ |
| Lua | | Aug 4, 2026 12:55a.m. | Review ↗ |
| Java | | Aug 4, 2026 12:55a.m. | Review ↗ |
| Go | | Aug 4, 2026 12:55a.m. | Review ↗ |
| C & C++ | | Aug 4, 2026 12:55a.m. | Review ↗ |
| Ansible | | Aug 4, 2026 12:55a.m. | Review ↗ |
| Apex | | Aug 4, 2026 12:55a.m. | Review ↗ |
| Elixir | | Aug 4, 2026 12:55a.m. | Review ↗ |
| Groovy | | Aug 4, 2026 12:55a.m. | Review ↗ |
| Objective-C | | Aug 4, 2026 12:55a.m. | Review ↗ |
| PowerShell | | Aug 4, 2026 12:55a.m. | Review ↗ |
| Terraform | | Aug 4, 2026 12:55a.m. | Review ↗ |
| VB.NET | | Aug 4, 2026 12:55a.m. | Review ↗ |
| SQL | | Aug 4, 2026 12:55a.m. | Review ↗ |
| Scala | | Aug 4, 2026 12:55a.m. | Review ↗ |
| Perl | | Aug 4, 2026 12:55a.m. | Review ↗ |
| Kotlin | | Aug 4, 2026 12:55a.m. | Review ↗ |
| Helm | | Aug 4, 2026 12:55a.m. | Review ↗ |
| Erlang | | Aug 4, 2026 12:55a.m. | Review ↗ |
| Dart | | Aug 4, 2026 12:55a.m. | Review ↗ |
| C# | | Aug 4, 2026 12:55a.m. | Review ↗ |
Important
AI Review is run only on demand for your team. We're only showing results of static analysis review right now. To trigger AI Review, comment @deepsourcebot review on this thread.
There was a problem hiding this comment.
Actionable comments posted: 2
🧹 Nitpick comments (1)
packages/opencode/test/provider/transform.test.ts (1)
684-712: 🎯 Functional Correctness | 🔵 Trivial | ⚡ Quick winAdd a regression test for the lower version boundary.
These tests do not cover GPT-5.4 with
useCompletionUrls: true. That case must still returnreasoningEffort: "medium"from Lines 1278-1279. A regression that removes the option for every completion model would pass the current tests.+ test("gpt-5.4 should set reasoningEffort for the completions API", () => {+ const result = ProviderTransform.options({+ model: createModel("gpt-5.4"),+ sessionID,+ providerOptions: { useCompletionUrls: true },+ })++ expect(result.reasoningEffort).toBe("medium")+ })🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In `@packages/opencode/test/provider/transform.test.ts` around lines 684 - 712, Add a regression test alongside the existing GPT-5.x completion tests in ProviderTransform.options covering model "gpt-5.4" with useCompletionUrls enabled, and assert that reasoningEffort remains "medium" as defined by the existing lower-version boundary behavior.
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@packages/console/core/src/referral.ts`:
- Around line 62-85: Update the referral reward query’s AuthTable join in the
rewards retrieval flow to use a left join instead of an inner join, and make
inviteeEmail nullable so active rewards remain included when email
authentication records are absent.
In `@packages/opencode/src/provider/transform.ts`:
- Around line 1277-1280: Update the final Azure options cleanup in the transform
flow around the `useCompletionUrls` handling so GPT-5.5+ completion requests
remove the previously added `reasoningEffort` along with the other unsupported
fields. Ensure this uses the merged model, agent, and variant options rather
than only the initial provider options, and add coverage for each configuration
layer.
---
Nitpick comments:
In `@packages/opencode/test/provider/transform.test.ts`:
- Around line 684-712: Add a regression test alongside the existing GPT-5.x
completion tests in ProviderTransform.options covering model "gpt-5.4" with
useCompletionUrls enabled, and assert that reasoningEffort remains "medium" as
defined by the existing lower-version boundary behavior.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Repository UI
Review profile: CHILL
Plan: Pro Plus
Run ID: 2d25fc88-f7b9-487e-95d8-2c3d23cc51ae
📒 Files selected for processing (7)
packages/console/core/src/referral.tspackages/opencode/src/provider/transform.tspackages/opencode/test/provider/transform.test.tspackages/session-ui/src/components/file.tsxpackages/session-ui/src/pierre/index.tspackages/session-ui/src/v2/components/line-comment-annotations-v2.tsxpackages/ui/src/theme/themes/oc-2.json
| const [rewards, invites, inviteeReferral, inviteeRewards] = await Promise.all([ | ||
| tx | ||
| .select({ | ||
| referralID: ReferralRewardTable.referralID, | ||
| workspaceID: ReferralRewardTable.workspaceID, | ||
| referralWorkspaceID: ReferralTable.workspaceID, | ||
| inviteeEmail: AuthTable.subject, | ||
| amount: ReferralRewardTable.amount, | ||
| timeCreated: ReferralRewardTable.timeCreated, | ||
| timeApplied: ReferralRewardTable.timeApplied, | ||
| }) | ||
| .from(ReferralRewardTable) | ||
| .innerJoin(ReferralTable, eq(ReferralTable.id, ReferralRewardTable.referralID)) | ||
| .innerJoin( | ||
| AuthTable, | ||
| and(eq(AuthTable.accountID, ReferralTable.inviteeAccountID), eq(AuthTable.provider, "email")), | ||
| ) | ||
| .where( | ||
| and( | ||
| eq(ReferralRewardTable.workspaceID, workspaceID), | ||
| isNull(ReferralRewardTable.timeDeleted), | ||
| isNull(ReferralTable.timeDeleted), | ||
| ), | ||
| ), |
There was a problem hiding this comment.
🗄️ Data Integrity & Integration | 🟠 Major | ⚡ Quick win
🧩 Analysis chain
🏁 Script executed:
printf'%s\n''--- file outline ---'
ast-grep outline packages/console/core/src/referral.ts --view compact
printf'%s\n''--- relevant source ---'
sed -n '1,155p' packages/console/core/src/referral.ts
sed -n '350,465p' packages/console/core/src/referral.ts
printf'%s\n''--- table definitions and AuthTable constraints ---'
rg -n -C 4 'ReferralRewardTable|ReferralTable|AuthTable|provider.*email|email.*provider' packages/console packages -g '*.ts' -g '*.tsx'| head -n 500Repository: bolt-builder/bolt-cli
Length of output: 50377
🏁 Script executed:
python3 - <<'PY'from pathlib import Pathp = Path("packages/console/core/src/referral.ts")lines = p.read_text().splitlines()for start, end in [(55, 145), (370, 410), (430, 455)]: print(f"--- {p}:{start}-{end} ---") for n in range(start, min(end, len(lines)) + 1): print(f"{n}:{lines[n-1]}")PYprintf'%s\n''--- exact schema file candidates ---'
fd -i 'schema|referral|auth' packages/console packages -t f | head -n 200Repository: bolt-builder/bolt-cli
Length of output: 11884
🏁 Script executed:
printf'%s\n''--- AuthTable schema ---'
cat -n packages/console/core/src/schema/auth.sql.ts
printf'%s\n''--- Referral schemas ---'
cat -n packages/console/core/src/schema/referral.sql.ts
printf'%s\n''--- account/user schemas ---'
cat -n packages/console/core/src/schema/account.sql.ts
cat -n packages/console/core/src/schema/user.sql.ts | sed -n '1,120p'printf'%s\n''--- AuthTable writes and referral.create callers ---'
rg -n -C 3 'insert\(AuthTable\)|insert\([^)]*AuthTable|Referral\.create|Referral\.summary' packages/console/core packages/console/app packages/console/function -g '*.ts' -g '*.tsx'| head -n 500Repository: bolt-builder/bolt-cli
Length of output: 8115
🏁 Script executed:
python3 - <<'PY'from pathlib import Pathfor name in [ "packages/console/core/src/schema/auth.sql.ts", "packages/console/core/src/schema/referral.sql.ts", "packages/console/core/src/schema/account.sql.ts",]: p = Path(name) print(f"--- {name} ---") for n, line in enumerate(p.read_text().splitlines(), 1): print(f"{n}:{line}")PYprintf'%s\n''--- all referral summary consumers ---'
rg -n -C 5 'Referral\.summary|summary\(' packages/console/app packages/console/core packages/console/function -g '*.ts' -g '*.tsx'| rg -C 5 'Referral|referral'| head -n 300Repository: bolt-builder/bolt-cli
Length of output: 5883
🏁 Script executed:
python3 - <<'PY'from pathlib import Pathfor name, ranges in { "packages/console/function/src/auth.ts": [(130, 205)], "packages/console/core/src/referral.ts": [(260, 365), (360, 470)], "packages/console/app/src/lib/referral-invite.ts": [(1, 55)],}.items(): lines = Path(name).read_text().splitlines() for start, end in ranges: print(f"--- {name}:{start}-{end} ---") for n in range(start, min(end, len(lines)) + 1): print(f"{n}:{lines[n-1]}")PYprintf'%s\n''--- read-only invariant probe ---'
python3 - <<'PY'from pathlib import Pathreferral = Path("packages/console/core/src/referral.ts").read_text()auth = Path("packages/console/core/src/schema/auth.sql.ts").read_text()user = Path("packages/console/core/src/schema/user.sql.ts").read_text()assert '.innerJoin(\n AuthTable,' in referralassert 'provider = "email"' in referralassert 'ReferralRewardTable' in referral and '.values([' in referralassert 'accountID: ulid("account_id")' in userassert 'provider: mysqlEnum("provider", AuthProvider).notNull()' in authassert 'uniqueIndex("provider").on(table.provider, table.subject)' in authprint("Reward creation does not join AuthTable.")print("AuthTable has no schema-level constraint requiring provider=email for every account.")print("UserTable.accountID is nullable.")print("The reward summary join can therefore remove a reward row when the invitee has no email auth row.")PYRepository: bolt-builder/bolt-cli
Length of output: 14771
🏁 Script executed:
printf'%s\n''--- every AuthTable write ---'
rg -n -C 6 'insert\(AuthTable\)|AuthTable\)\.values|AuthTable\)\.ignore|update\(AuthTable\)|delete\(AuthTable\)' packages/console -g '*.ts' -g '*.tsx'printf'%s\n''--- every account creation path ---'
rg -n -C 8 'Account\.create|insert\(AccountTable\)|AccountTable\)\.values' packages/console -g '*.ts' -g '*.tsx'printf'%s\n''--- auth row deletion and account/user creation ---'
rg -n -C 5 'timeDeleted.*AuthTable|AuthTable.*timeDeleted|UserTable.*accountID|accountID.*UserTable' packages/console/core packages/console/function packages/console/app -g '*.ts' -g '*.tsx'| head -n 500Repository: bolt-builder/bolt-cli
Length of output: 30710
🏁 Script executed:
python3 - <<'PY'from pathlib import Pathfor name in [ "packages/console/core/src/account.ts", "packages/console/core/src/user.ts",]: p = Path(name) print(f"--- {name} ---") lines = p.read_text().splitlines() for n, line in enumerate(lines, 1): if any(token in line for token in ("Account.create", "AccountTable", "AuthTable", "UserTable")): lo, hi = max(1, n - 5), min(len(lines), n + 12) print(f"[{lo}-{hi}]") for i in range(lo, hi + 1): print(f"{i}:{lines[i-1]}")PYRepository: bolt-builder/bolt-cli
Length of output: 35982
Do not make active reward history depend on email authentication.
Account.remove deletes AuthTable rows but leaves active referrals and rewards. This innerJoin then hides those rewards from the summary. Use leftJoin and keep inviteeEmail nullable.
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
In `@packages/console/core/src/referral.ts` around lines 62 - 85, Update the
referral reward query’s AuthTable join in the rewards retrieval flow to use a
left join instead of an inner join, and make inviteeEmail nullable so active
rewards remain included when email authentication records are absent.
| if (input.model.api.npm === "@ai-sdk/azure" && input.providerOptions?.useCompletionUrls) { | ||
| if (!isGpt55OrNewer) { | ||
| result["reasoningEffort"] = "medium" | ||
| } |
There was a problem hiding this comment.
🗄️ Data Integrity & Integration | 🟠 Major | ⚡ Quick win
🧩 Analysis chain
🏁 Script executed:
#!/usr/bin/env bashset -euo pipefail
rg -n -C 6 \
'ProviderTransform\.options|useCompletionUrls|reasoningEffort|reasoningSummary|model\.options|agent\.options|variant|mergeOptions' \
packages/opencode/src packages/opencode/testRepository: bolt-builder/bolt-cli
Length of output: 50378
🏁 Script executed:
#!/usr/bin/env bashset -euo pipefail
printf'%s\n''--- transform.ts relevant symbols ---'
rg -n -C 18 'function options|options\(|useCompletionUrls|isGpt55OrNewer|reasoningEffort' packages/opencode/src/provider/transform.ts | head -n 260
printf'%s\n''--- request path references ---'
rg -n -C 20 'ProviderTransform\.options|useCompletionUrls|providerOptions|modelOptions|agent|variant|reasoningEffort' packages/opencode/src/session/llm/request.ts | head -n 320
printf'%s\n''--- candidate option merge definitions ---'
rg -n -C 12 'merge.*option|options.*merge|providerOptions|model\.options|agent\.options|variant' packages/opencode/src/session packages/opencode/src/provider | head -n 320Repository: bolt-builder/bolt-cli
Length of output: 42857
🏁 Script executed:
#!/usr/bin/env bashset -euo pipefail
printf'%s\n''--- Azure options implementation ---'
sed -n '1180,1310p' packages/opencode/src/provider/transform.ts
printf'%s\n''--- Azure completion-url tests and request-preparation tests ---'
rg -n -C 14 'useCompletionUrls|completion.?url|Azure|azure' \
packages/opencode/test/provider/transform.test.ts \
packages/opencode/test/session packages/opencode/test 2>/dev/null | head -n 360
printf'%s\n''--- request and transform status ---'
git status --short -- packages/opencode/src/provider/transform.ts packages/opencode/src/session/llm/request.ts packages/opencode/test/provider/transform.test.ts
git diff --stat -- packages/opencode/src/provider/transform.ts packages/opencode/src/session/llm/request.ts packages/opencode/test/provider/transform.test.tsRepository: bolt-builder/bolt-cli
Length of output: 34978
🏁 Script executed:
#!/usr/bin/env bashset -euo pipefail
printf'%s\n''--- GPT-5 Azure transform tests ---'
sed -n '626,716p' packages/opencode/test/provider/transform.test.ts
printf'%s\n''--- Existing merged-options regression test ---'
sed -n '510,592p' packages/opencode/test/provider/transform.test.ts
printf'%s\n''--- Exact cleanup implementation ---'
sed -n '78,101p' packages/opencode/src/session/llm/request.ts
printf'%s\n''--- Read-only behavioral verifier for the relevant branches ---'
python3 - <<'PY'def transform_options(api_id, provider_options): result = {} major_minor = api_id.removeprefix("gpt-").split(".", 1) major = int(major_minor[0]) if len(major_minor) == 2 else None minor = int(major_minor[1]) if len(major_minor) == 2 else None gpt55_or_newer = major is not None and (major > 5 or (major == 5 and minor >= 5)) if provider_options.get("useCompletionUrls"): if not gpt55_or_newer: result["reasoningEffort"] = "medium" return result if "gpt-5" in api_id and "gpt-5-chat" not in api_id and "gpt-5-pro" not in api_id: result["reasoningEffort"] = "medium" result["reasoningSummary"] = "auto" result["include"] = ["reasoning.encrypted_content"] return resultdef prepare(api_id, provider_options, model_options, agent_options, variant): base = transform_options(api_id, provider_options) options = {**base, **model_options, **agent_options, **variant} if ( provider_options.get("useCompletionUrls") or model_options.get("useCompletionUrls") or options.get("useCompletionUrls") ): options.pop("reasoningSummary", None) options.pop("include", None) return optionscases = { "provider": prepare("gpt-5.5", {"useCompletionUrls": True}, {}, {}, {}), "model": prepare("gpt-5.5", {}, {"useCompletionUrls": True}, {}, {}), "agent": prepare("gpt-5.5", {}, {}, {"useCompletionUrls": True}, {}), "variant": prepare("gpt-5.5", {}, {}, {}, {"useCompletionUrls": True}),}for name, result in cases.items(): print(name, result)assert cases["provider"] == {}assert all(result.get("reasoningEffort") == "medium" for name, result in cases.items() if name != "provider")PYRepository: bolt-builder/bolt-cli
Length of output: 6392
Handle useCompletionUrls in the final Azure options.
When useCompletionUrls comes from model, agent, or variant options, the initial transform still adds reasoningEffort: "medium" for GPT-5.5+ models. The post-merge cleanup removes only reasoningSummary and include. Remove reasoningEffort for GPT-5.5+ completion requests, or pass the merged options into the transform. Add coverage for each configuration layer.
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
In `@packages/opencode/src/provider/transform.ts` around lines 1277 - 1280, Update
the final Azure options cleanup in the transform flow around the
`useCompletionUrls` handling so GPT-5.5+ completion requests remove the
previously added `reasoningEffort` along with the other unsupported fields.
Ensure this uses the merged model, agent, and variant options rather than only
the initial provider options, and add coverage for each configuration layer.
A-x6
commented
Aug 4, 2026
@codesmith-bot fix merge confliicts and then push to this pr not a another pr but dont merge and tell me does this bring any ui branding? and explain to me everything else in this repo |
Got it, I'm checking the PR diff and merge conflicts first. |
![View with [code]smith](https://pr-comments-assets.blacksmith.sh/codesmith/view-with-codesmith-dark-v2.svg)
@DevFlex-AI
Done. Here's everything: The conflicts and where the fix lives. All 29 conflicted files were the same trivial pattern: your fork's Why I couldn't push to PR #150 itself. This PR's head branch is Does this bring any UI branding? No. The merge is only 5 upstream commits touching 7 files, and none of them reintroduce OpenCode names, logos, or branding. What it actually brings: referral reward history in What this repo is.
Dependency rules flow Schema → Core/Protocol → Server, with clients only touching Schema/Protocol. Conventional commits, |
Uh oh!
There was an error while loading. Please reload this page.
Docs PR opened: bolt-builder/docs#2 Added a changelog entry for the v1.18.12 release covering GPT-5 reasoning tiers, diff polish, and selection fixes. |
Issue for this PR
Closes #
Type of change
What does this PR do?
Please provide a description of the issue, the changes you made to fix it, and why they work. It is expected that you understand why your changes work and if you do not understand why at least say as much so a maintainer knows how much to value the PR.
If you paste a large clearly AI generated description here your PR may be IGNORED or CLOSED!
How did you verify your code works?
Screenshots / recordings
If this is a UI change, please include a screenshot or recording.
Checklist
If you do not follow this template your PR will be automatically rejected.
Need help on this PR? Tag
@codesmith-botwith what you need. Autofix is disabled.Summary by CodeRabbit
New Features
Bug Fixes
Chores