Skip to content

feat(web): expose bounded runtime diagnostics - #367

Open
seekskyworld wants to merge 4 commits into
openpi-dev:mainfrom
seekskyworld:feat/issue-348-runtime-diagnostics
Open

feat(web): expose bounded runtime diagnostics#367
seekskyworld wants to merge 4 commits into
openpi-dev:mainfrom
seekskyworld:feat/issue-348-runtime-diagnostics

Conversation

@seekskyworld

Copy link
Copy Markdown

Problem

Implements the read-only diagnostics slice of #348. Web users had no bounded way to inspect the active runtime, Session identity, workspace selection, or current model.

Value

Support can distinguish runtime and Session configuration without exposing credentials or transcript data.

Approach

Add authenticated GET /api/diagnostics returning Node version, canonical runtime cwd, active Session id, workspace selection, and current model summaries. Provider credentials and secrets remain outside the projection.

Validation

  • npx tsc --noEmit
  • git diff --check

Impact

  • User-visible behavior: adds a bounded diagnostics response for the Web workbench.
  • Model-visible context/tools: none.
  • Runtime/lifecycle: read-only inspection.
  • Persisted config/data: none.
  • Compatibility/risk: additive endpoint; provider login and setup episode remain separate work.

Signed-off-by: seekskyworld <djh1813553759@gmail.com>
Signed-off-by: seekskyworld <djh1813553759@gmail.com>
Prove unauthenticated rejection, read-only projection, and bounded
fields so GET /api/capabilities and /api/diagnostics cannot silently
grow secrets or transcript evidence.

@tt-a1i tt-a1i left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Approved at exact head 3bd6dfc9ecf2604c42c59c8f78356d8a48e7eeb1. I independently reviewed the scoped implementation and boundary conditions, verified the current diff against main, and found no unresolved P0/P1 issues. Required CI checks are green at this head.

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