Repository navigation
Conversation
|
Important Draft PR not reviewedDraft PRs are not automatically reviewed by default.
To automatically review draft PRs, update your CodeRabbit configuration: reviews:
auto_review:
drafts: true
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 |
|
⏳ DRAFT
What to do
Review readiness checklist
2/4 boxes ticked. This pull request was already a draft. Its draft status will be preserved after every issue above is resolved. |
|
@lidge-jun This draft is blocked by @Ingwannu Both failed checks ( |
|
For the explicit review request: I confirmed head Cross-platform CI |
Summary
compaction_triggerand v1/responses/compact. Custom gateways retain portable summaries by default. Generation capability does not imply ciphertext replay, OpenAI identity or permission to forward caller credentials.Verification
b93c76f936fa6a2c8c2ab928d2dca60d521b61ee; contained/current dev:7af092ca4f78828238b425bf351d5989e237f89f. Original contributor commit61afaab15b5302dc10a954725bd2e14c69cee34eremains an ancestor. Normal push to the existing contributor branch; no force push/rebase.OCX_TEST_NO_QUEUE=1 bun run scripts/test.ts --parallel=2 tests/responses/native-compaction-oauth.test.ts tests/responses/native-compaction-provenance.test.ts tests/responses/native-compaction-gateway.test.ts tests/responses/native-compaction-gateway-policy.test.ts tests/responses/responses-compaction.test.ts tests/responses/responses-compaction-routing.test.ts tests/responses/responses-compaction-policy-identity.test.ts tests/responses/responses-compaction-override.test.ts tests/responses/responses-compaction-recovery.test.ts tests/responses/responses-compaction-recovery-policy.test.ts tests/responses/responses-canonical-only-top-level-fields.test.ts tests/responses/responses-opaque-blob-recovery.test.ts tests/responses/opaque-blob-wrapped-rejection.test.ts tests/adapters/reasoning-replay-identity.test.ts tests/adapters/reasoning-replay-robustness.test.ts tests/lib/reasoning-replay-scope-source.test.ts tests/providers/deepseek-reasoning-replay.test.ts: 636 pass / 0 fail, 17 files.bun run typecheck,bun run structure:check,bun run privacy:scan: exit 0. Layout/tooling/ratchet/Lab guard batch initially 52 pass / 1 fail only for new seed classification. Corrected both seed regexes; final layout/tooling 19 pass / 0 fail. Ratchet 9 and Lab 25 passed before the seed-only correction; caps unchanged and both inventories registered.bun run --cwd docs-site build: exit 0, 569 pages / 79,338 internal links. No dependency declarations/lockfiles changed.b93c76f936fa6a2c8c2ab928d2dca60d521b61eeagainst dev7af092ca4f78828238b425bf351d5989e237f89f: all 30 changed files reviewed, no concrete P0–P2 findings; 168 pure in-memory checks passed with zero network sends. Explicit maintainer security/merge disposition remains with the coordinator.Checklist
Review readiness checklist