Conversation
|
Important Review skippedToo many files! This PR contains 336 files, which is 236 over the limit of 100. To get a review, reduce the PR to 100 files or fewer by splitting it into smaller PRs or changing its base branch. Upgrade to a paid plan to raise the limit. Usage-priced reviews support at most 300 files. ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Advanced Run ID: ⛔ Files ignored due to path filters (2)
📒 Files selected for processing (336)
You can disable this status message by setting the Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Gavel summary
Totals: 0 passed · 0 failed · 0 skipped · - |
|
Review the following changes in direct dependencies. Learn more about Socket for GitHub.
|
Gavel summary
Totals: 5724 passed · 14 failed · 12 skipped · 3m45s Failing testscaptain — runtime entity routes > over the executor > creates, lists, updates and resolves through the published routesmigrations — Tool approval identity migration > backfills an unambiguous legacy approval and replaces the credential constraintmigrations — Tool approval identity migration > accepts a provider approval with no credential, turn or model callmigrations — Tool approval identity migration > still rejects a caller-tool approval that names no turnmigrations — Tool approval identity migration > re-applies the retired 74 and its successor with a provider approval already stored... and 9 more failing tests — see the full gavel-results artifact. |
Claude-Session-Id: c1750ebd-c2e1-43bc-8962-5499fa2e65d7
A decoded `"permissions": {}` marked /permissions explicitly present, and
the explicit-path pass then overwrote the merged section with the zero
struct — dropping every inherited value the override never named, so a
request that merely mentioned the section erased the project's tool policy.
An explicit path naming an all-zero struct now merges field-wise instead of
replacing. Every value the override actually supplied inside the section
still carries its own explicit path and still wins, including a
deliberately emptied collection like `"tools": {}`.
Claude-Session: https://claude.ai/code/session_01QemSU28zWbxmrRuNKfoFPb
Claude-Session-Id: 2d8cf8c1-0626-49fd-b02b-654e7c1554fe
…s only A spec layer is a source of defaults: the last layer naming a field owns it, and no layer restricts what a later one may select. A posture that must survive the whole stack is applied by its owner as the final layer rather than hung off an earlier layer as a ceiling. Deletes RuntimeConstraints and everything that folded it — the permission ceiling lattice, the restrictive model catalog, run limits and usage quotas, ValidateRuntimeConstraints, and the aichat/promptrun enforcement paths that read them. Nothing outside tests ever produced a model or quota constraint, and the permission lattice ranked only four of the six permission modes, so a stack pairing an unranked posture (auto, dontAsk) with any other refused to compose at all — one such pairing took down a whole run-context response. ComposeSpecLayers is now the fold alone: order the layers, merge the specs, record provenance, validate the effective budget. Claude-Session: https://claude.ai/code/session_01QemSU28zWbxmrRuNKfoFPb Claude-Session-Id: 2d8cf8c1-0626-49fd-b02b-654e7c1554fe
…nventory and Go DTOs Claude-Session-Id: 01a08ec3-abce-7663-b1a3-546c0b710945
Claude-Session-Id: 01a09053-85e4-7382-9b30-5a4048a9c95f
Claude-Session-Id: d72dec9c-ff9c-4f1e-b821-264063be5727
Claude-Session-Id: c7ef9a1a-c38c-4d0e-bf53-db8d1829d974
…nt mapper config Claude-Session-Id: 01a0a813-4df1-72f0-b045-3c31d672c8c0
…t-in defaults Claude-Session-Id: 01a0a813-4df1-72f0-b045-3c31d672c8c0
Claude-Session-Id: 01a0a813-4df1-72f0-b045-3c31d672c8c0
…n chat Claude-Session-Id: 01a0a813-4df1-72f0-b045-3c31d672c8c0
Claude-Session-Id: 01a0a813-4df1-72f0-b045-3c31d672c8c0
Claude-Session-Id: 01a0a813-4df1-72f0-b045-3c31d672c8c0
Claude-Session-Id: 01a0b3ab-29e0-7ee1-aa0e-89f7b48065a7
Claude-Session-Id: fe63c69c-4e6a-4230-86da-fca899327996
Claude-Session-Id: 01a0cc87-ee7d-7721-baca-11e38c8badba
Claude-Session-Id: c7d6497a-8a15-4f2f-b203-32fa7f360cac
No description provided.