Skip to content

backup:\ pre-wave\ full\ dirty\ snapshot\ before\ fresh-main\ worktree\ execution#410

Merged
KooshaPari merged 1 commit intomainfrom
pr/cliproxy-layer1-pkg
Feb 25, 2026
Merged

backup:\ pre-wave\ full\ dirty\ snapshot\ before\ fresh-main\ worktree\ execution#410
KooshaPari merged 1 commit intomainfrom
pr/cliproxy-layer1-pkg

Conversation

@KooshaPari
Copy link
Copy Markdown
Owner

@KooshaPari KooshaPari commented Feb 25, 2026

Automated PR for pr/cliproxy-layer1-pkg branch state sync.

Summary by CodeRabbit

  • New Features

    • Provider error runbook for operational troubleshooting scenarios
    • OAuth callback and headless authentication flows
    • Provider-specific proxy override configuration
    • GPT-5 Pro model support with extended thinking capability
    • Droid provider alias normalization
    • Configurable responses compact toggle
  • Documentation

    • Expanded operational runbooks and troubleshooting guides
    • Added provider-specific setup and authentication documentation
    • Enhanced error triage matrices with remediation steps

@gemini-code-assist
Copy link
Copy Markdown

Summary of Changes

Hello @KooshaPari, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed!

This pull request primarily focuses on synchronizing the pr/cliproxy-layer1-pkg branch state, incorporating a new configuration toggle for OpenAI-compatible compact responses, refining provider alias handling for improved telemetry, and substantially enhancing the project's documentation. The updates provide more comprehensive operational runbooks, detailed planning reports, and expanded troubleshooting guides to support various provider integrations and setup scenarios.

Highlights

  • Configuration Toggle for Compact Responses: Introduced a new configuration option, responses-compact-enabled, in config.example.yaml to gate the OpenAI-compatible /v1/responses/compact behavior, allowing for staged rollout or rapid disabling. The default behavior is enabled when omitted.
  • Enhanced Provider Alias Handling: Tightened assertions for provider_alias and provider_aliased in cliproxyctl login provider tests and added a test to ensure 'droid' aliases correctly map to 'gemini' in usage metrics, improving consistency and telemetry.
  • Expanded Operational Runbooks: Added a new Provider Error Runbook Snippets document to docs/operations/index.md, providing actionable runbook entries for specific Huggingface and Codex backend-api errors, including validation commands and remediation steps.
  • Updated Planning and Triage Documentation: Significantly updated various planning and troubleshooting documents (docs/planning/reports/*, docs/provider-operations.md, docs/provider-quickstarts.md, docs/troubleshooting.md) to reflect implemented issues, new validation commands, and detailed guidance for provider-specific proxy overrides, OAuth callback flows, and model behavior.
  • Code Logic for Compact Responses: Implemented a check in OpenAICompatExecutor to prevent the /responses/compact endpoint from being used if it is explicitly disabled in the configuration, ensuring adherence to the new toggle.
Changelog
  • cmd/cliproxyctl/main_test.go
    • Added assertions for provider_alias and provider_aliased to the TestResolveLoginProviderNormalizesDroidAliases test.
  • config.example.yaml
    • Added a commented-out entry for responses-compact-enabled to control the /v1/responses/compact behavior.
  • docs/operations/index.md
    • Linked the new Provider Error Runbook Snippets document.
  • docs/operations/provider-error-runbook.md
    • Added a new document detailing runbook snippets for Huggingface CLIProxyAPI errors (CPB-0803) and Codex backend-api Not Found errors (CPB-0804).
  • docs/planning/reports/issue-wave-cpb-0781-0830-implementation-batch-3-resume-12.md
    • Added a new report detailing the implementation of several CPB issues related to documentation and runbook hardening.
  • docs/planning/reports/issue-wave-cpb-0781-0830-implementation-batch-4-code.md
    • Updated the report to include new implemented IDs (CPB-0819, CPB-0820, CPB-0821) and corresponding file changes and validation commands.
  • docs/planning/reports/issue-wave-cpb-0781-0830-lane-b.md
    • Updated with 'Execution Update (Batch 3)' details for CPB-0789 to CPB-0796, focusing on quickstart/troubleshooting documentation.
  • docs/planning/reports/issue-wave-cpb-0781-0830-lane-c.md
    • Updated with 'Execution Update (Batch 3)' details for CPB-0797 to CPB-0804, focusing on quickstart/troubleshooting documentation and provider error runbooks.
  • docs/planning/reports/issue-wave-cpb-0781-0830-next-50-summary.md
    • Updated the summary of implemented issues from 16/50 to 28/50.
    • Added a 'count basis' note for resume-scoped verification.
    • Updated the 'Implemented in this batch' section to include CPB-0819, CPB-0820, CPB-0821 and the 'Resume Scoped 12' batch.
  • docs/provider-operations.md
    • Added a new section on 'Provider-specific proxy overrides (CPB-0794)' with configuration knobs and validation steps.
  • docs/provider-quickstarts.md
    • Updated sections on 'Reasoning parity probe', 'Prompt cache guardrails', 'Compose health check', and 'Proxy/auth/usage checks' with more detailed validation commands and expectations.
    • Added new sections for 'Antigravity cliproxyctl flow (CPB-0798)' and 'Manual callback headless OAuth (CPB-0800)'.
  • docs/troubleshooting.md
    • Added new entries to the troubleshooting matrix for various issues, including Kiro quota, Gemini CLI OAuth callback failures, auth-dir permissions, login/runtime auth-dir mismatches, and several CPB-prefixed issues related to specific model behaviors and streaming.
    • Added new sections for 'OAuth Callback Server Start Failure (Gemini/Antigravity)', 'Wave Batch 2 triage matrix', and 'Wave Batch 3 triage matrix'.
  • pkg/llmproxy/config/responses_compact_toggle_test.go
    • Added a new test file for the IsResponsesCompactEnabled function, testing default behavior and explicit toggle.
  • pkg/llmproxy/executor/openai_compat_executor.go
    • Added a check to disable the /responses/compact endpoint if ResponsesCompactEnabled is false in the configuration.
  • pkg/llmproxy/executor/openai_compat_executor_compact_test.go
    • Added a test case to verify that the /responses/compact endpoint returns StatusNotFound when disabled by configuration.
  • pkg/llmproxy/usage/metrics_test.go
    • Added a test case TestGetProviderMetrics_MapsDroidAliasToGemini to ensure 'droid' alias maps to 'gemini' in provider metrics.
Activity
  • This pull request was automatically generated to synchronize the branch state of pr/cliproxy-layer1-pkg.
Using Gemini Code Assist

The full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips.

Invoking Gemini

You can request assistance from Gemini at any point by creating a comment using either /gemini <command> or @gemini-code-assist <command>. Below is a summary of the supported commands on the current page.

Feature Command Description
Code Review /gemini review Performs a code review for the current pull request in its current state.
Pull Request Summary /gemini summary Provides a summary of the current pull request in its current state.
Comment @gemini-code-assist Responds in comments when explicitly tagged, both in pull request comments and review comments.
Help /gemini help Displays a list of available commands.

Customization

To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a .gemini/ folder in the base of the repository. Detailed instructions can be found here.

Limitations & Feedback

Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for Github and other Google products, sign up here.

You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension.

Footnotes

  1. Review the Privacy Notices, Generative AI Prohibited Use Policy, Terms of Service, and learn how to configure Gemini Code Assist in GitHub here. Gemini can make mistakes, so double check it and use code with caution.

@KooshaPari KooshaPari merged commit 83c6fb2 into main Feb 25, 2026
2 of 6 checks passed
Copy link
Copy Markdown

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request appears to be an automated state sync, bringing in a wide range of changes. The bulk of the updates are to documentation, including new runbooks, quickstart guides, and planning reports, which enhance operational readiness and developer guidance. Key code changes include the implementation of a feature toggle for /v1/responses/compact behavior, complete with tests to ensure it works as expected. Additionally, tests for provider aliasing (e.g., droid to gemini) have been strengthened. My review found a couple of minor formatting improvement opportunities in the documentation.


### Antigravity cliproxyctl flow (`CPB-0798`)

```
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

medium

For consistency with other code blocks in this document and to enable syntax highlighting, please specify the language for this code block.

Suggested change
```
```bash


### Manual callback headless OAuth (`CPB-0800`)

```
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

medium

For consistency with other code blocks in this document and to enable syntax highlighting, please specify the language for this code block.

Suggested change
```
```bash

@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Feb 25, 2026

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info

Configuration used: defaults

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 5b02096 and bd8a546.

📒 Files selected for processing (16)
  • cmd/cliproxyctl/main_test.go
  • config.example.yaml
  • docs/operations/index.md
  • docs/operations/provider-error-runbook.md
  • docs/planning/reports/issue-wave-cpb-0781-0830-implementation-batch-3-resume-12.md
  • docs/planning/reports/issue-wave-cpb-0781-0830-implementation-batch-4-code.md
  • docs/planning/reports/issue-wave-cpb-0781-0830-lane-b.md
  • docs/planning/reports/issue-wave-cpb-0781-0830-lane-c.md
  • docs/planning/reports/issue-wave-cpb-0781-0830-next-50-summary.md
  • docs/provider-operations.md
  • docs/provider-quickstarts.md
  • docs/troubleshooting.md
  • pkg/llmproxy/config/responses_compact_toggle_test.go
  • pkg/llmproxy/executor/openai_compat_executor.go
  • pkg/llmproxy/executor/openai_compat_executor_compact_test.go
  • pkg/llmproxy/usage/metrics_test.go

📝 Walkthrough

Walkthrough

This pull request implements feature-flag configuration for the responses/compact endpoint, adds provider alias normalization with comprehensive test coverage, and expands documentation across operations, troubleshooting, and planning reports with error runbooks, triage matrices, and implementation tracking.

Changes

Cohort / File(s) Summary
Responses Compact Config Toggle
pkg/llmproxy/config/responses_compact_toggle_test.go, pkg/llmproxy/executor/openai_compat_executor.go, pkg/llmproxy/executor/openai_compat_executor_compact_test.go, config.example.yaml
Introduces config-gated /v1/responses/compact endpoint with default enabled behavior. Executor checks config and returns 404 if disabled. Tests verify nil-pointer defaults, zero-value struct behavior, and explicit boolean toggling.
Provider Alias Normalization & Metrics
cmd/cliproxyctl/main_test.go, pkg/llmproxy/usage/metrics_test.go
Adds test assertions validating droid alias normalization to gemini provider. Tests verify provider_alias and provider_aliased fields, plus metrics aggregation from droid provider input to gemini output.
Operational Documentation
docs/operations/index.md, docs/operations/provider-error-runbook.md, docs/provider-operations.md
Adds provider error runbook with CPB-0803 and CPB-0804 scenarios including symptoms, validation commands, and remediation steps. Documents proxy override configuration and updates mitigation guidance.
Quickstarts & Troubleshooting
docs/provider-quickstarts.md, docs/troubleshooting.md
Expands quickstart coverage with Antigravity flow and manual OAuth sections, updates request/response examples emphasizing parity and usage metadata. Adds triage sections for gemini-3-pro tool-use, OAuth callback failures, and wave batch matrices.
Planning & Reporting
docs/planning/reports/issue-wave-cpb-0781-0830-*
Updates implementation tracking across multiple batch reports with new CPB items (CPB-0819, CPB-0820, CPB-0821), validation commands, execution snapshots, and queue metrics reflecting 28/50 implemented and 22/50 remaining.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~25 minutes

Poem

🐰 A compact response and aliases so fine,
Provider errors now have their own line,
With droid mapping to gemini's glow,
And runbooks to help when things go slow,
Feature flags bloom as our docs dance and grow! 🌱

✨ Finishing Touches
  • 📝 Generate docstrings (stacked PR)
  • 📝 Generate docstrings (commit on current branch)
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch pr/cliproxy-layer1-pkg

Warning

There were issues while running some tools. Please review the errors and either fix the tool's configuration or disable the tool if it's a critical failure.

🔧 golangci-lint (2.5.0)

Error: can't load config: the Go language version (go1.25) used to build golangci-lint is lower than the targeted Go version (1.26.0)
The command is terminated due to an error: can't load config: the Go language version (go1.25) used to build golangci-lint is lower than the targeted Go version (1.26.0)


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.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

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