Skip to content

feat(diagnostics): expose structured diagnostics for Python supervisors#87

Open
SequeI wants to merge 1 commit into
mainfrom
exposeDiags
Open

feat(diagnostics): expose structured diagnostics for Python supervisors#87
SequeI wants to merge 1 commit into
mainfrom
exposeDiags

Conversation

@SequeI

@SequeI SequeI commented Jun 16, 2026

Copy link
Copy Markdown
Contributor

Add PyO3 bindings for nono #1171 structured diagnostics: error diagnostic_code and remediation attrs, proxy startup diagnostics, exec session reports with stderr translation, and merge helpers.

Waiting on nolabs-ai/nono#1171 to land first before this can be merged

Linked Issue

Closes #

Summary

Checklist

  • All commits are signed-off, using DCO
  • All new code has docstrings and type annotations
  • All new code is covered by tests
  • Public facing changes are paired with documentation changes
  • Type stubs (_nono_py.pyi) are updated if Rust API changed

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

Copy link
Copy Markdown

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 introduces structured diagnostics for Python clients by adding stderr parsing and diagnostic reporting capabilities, exposing new Python bindings such as session_diagnostics and diagnostics. The review feedback highlights critical compilation and compatibility issues: the invalid Python::attach API should be replaced with Python::with_gil, unstable let_chains in src/stderr_observation.rs must be rewritten using nested if let blocks for stable Rust compatibility, and the deprecated PyModule::import should be updated to py.import.

Important

The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.

Comment thread src/lib.rs
Comment thread src/proxy.rs
Comment thread src/sandboxed_exec.rs
Comment thread src/diagnostic.rs
Comment thread src/diagnostic.rs
Comment thread src/stderr_observation.rs
Comment thread src/stderr_observation.rs
Comment thread src/diagnostic.rs
Add PyO3 bindings for nono #1171 structured diagnostics: error diagnostic_code
and remediation attrs, proxy startup diagnostics, exec session reports with
stderr translation, and merge helpers.

Signed-off-by: Aleksy Siek <aleksy@alwaysfurther.ai>
@SequeI SequeI marked this pull request as ready for review June 22, 2026 09:17

@connrg connrg left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

lgtm

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.

2 participants