Skip to content

feat(agents): stream provider observations - #159

Open
Waishnav wants to merge 2 commits into
codex/workflow-navigator-inspectorfrom
codex/workflow-provider-adapters
Open

feat(agents): stream provider observations#159
Waishnav wants to merge 2 commits into
codex/workflow-navigator-inspectorfrom
codex/workflow-provider-adapters

Conversation

@Waishnav

@Waishnav Waishnav commented Aug 8, 2026

Copy link
Copy Markdown
Owner

Agent providers expose different event and token-usage shapes, so workflow observability otherwise stops at the adapter boundary. Structural normalizers now cover Codex, Claude, OpenCode, Pi, and ACP-backed Cursor/Copilot events, while each adapter forwards best-effort tool activity and usage into the workflow journal and CLI output. Values are only emitted when the provider supplies them; unsupported or disabled providers do not produce fabricated observations. This is the top layer of the stack.

@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.
To continue using code reviews, you can upgrade your account or add credits to your account and enable them for code reviews in your settings.

@coderabbitai

coderabbitai Bot commented Aug 8, 2026

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 5be2e928-c6ef-4313-a342-15cf504645a7

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

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.

@Waishnav Waishnav changed the title codex/workflow provider adapters feat(agents): stream provider observations Aug 8, 2026
@greptile-apps

greptile-apps Bot commented Aug 8, 2026

Copy link
Copy Markdown

Greptile Summary

The PR adds normalized provider activity and token-usage observations across Codex, Claude, OpenCode, Pi, and ACP adapters.

  • Introduces structural parsers for provider-specific tool activity and token usage payloads.
  • Emits observations from local-agent runtimes and exposes per-call and aggregate run usage in CLI output.
  • Adds provider parser and CLI usage coverage to the test suite.

Confidence Score: 5/5

The PR appears safe to merge, with no concrete blocking or independently actionable non-blocking defect established.

Provider observations flow through the existing workflow persistence path, usage fields are normalized and aggregated consistently with existing workflow views, and no reachable incorrect behavior remains supported by the available repository evidence.

Important Files Changed

Filename Overview
src/local-agent-provider-observations.ts Adds provider-independent structural extraction and normalization for tool activity and token usage.
src/local-agent-adapters.ts Connects Claude, OpenCode, ACP, and Pi provider payloads to observation emission and final usage reporting.
src/local-agent-runtime.ts Adds observation deduplication and Codex activity and usage extraction.
src/cli-output.ts Exposes per-call usage and field-wise aggregate workflow-run usage.
src/local-agent-provider-observations.test.ts Covers representative activity and usage payloads for all supported provider families.

Sequence Diagram

sequenceDiagram
    participant Provider
    participant Adapter
    participant Extractor as Observation Extractor
    participant Workflow as Workflow Runtime
    participant Store
    participant CLI
    Provider->>Adapter: events and final response
    Adapter->>Extractor: provider payloads
    Extractor-->>Adapter: activity and usage observations
    Adapter->>Workflow: observations and run result
    Workflow->>Store: persist activity and latest usage
    Store-->>CLI: call records
    CLI->>CLI: sum final call usage
Loading

Reviews (1): Last reviewed commit: "feat(agents): wire provider observation ..." | Re-trigger Greptile

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