Skip to content

feat(dashboard): show guard diagnostics#156

Merged
hasandemirkiran merged 1 commit into
mainfrom
feat/eng-328-dashboard-diagnostics
May 18, 2026
Merged

feat(dashboard): show guard diagnostics#156
hasandemirkiran merged 1 commit into
mainfrom
feat/eng-328-dashboard-diagnostics

Conversation

@hasandemirkiran
Copy link
Copy Markdown
Contributor

@hasandemirkiran hasandemirkiran commented May 18, 2026 •
edited
Loading

Summary

  • Adds clearer clicked-log diagnostics for guard decisions, including reason, stage, source, timestamp, policy version/profile/rule pack, policy rule details, and judge result/risk.
  • Removes user-facing local model details from dashboard logs and strips judge model metadata from dashboard event API responses.
  • Hides raw judge categories from the inspector because they are currently free-form model tags, not stable user-facing labels.

Verification

  • pnpm --dir web/guard-dashboard typecheck
  • scripts/build-guard-dashboard-assets.sh
  • go test ./...

Follow-up

  • Persist per-event observe/enforce mode so the inspector can show the exact mode for each decision.

@hasandemirkiran hasandemirkiran mentioned this pull request May 18, 2026
Merged
1 task
@hasandemirkiran Graphite App
Copy link
Copy Markdown
Contributor Author

hasandemirkiran commented May 18, 2026 •
edited
Loading

This stack of pull requests is managed by Graphite. Learn more about stacking.

@hasandemirkiran hasandemirkiran mentioned this pull request May 18, 2026
Merged
4 tasks
@hasandemirkiran hasandemirkiran force-pushed the feat/eng-328-dashboard-diagnostics branch from 50a2d7d to b574bcd Compare May 18, 2026 14:44
@hasandemirkiran hasandemirkiran marked this pull request as ready for review May 18, 2026 14:50
@chatgpt-codex-connector
Copy link
Copy Markdown

chatgpt-codex-connector Bot commented May 18, 2026

Codex usage limits have been reached for code reviews. Please check with the admins of this repo to increase the limits by adding credits.
Credits must be used to enable repository wide code reviews.

@hasandemirkiran hasandemirkiran requested a review from michiosw May 18, 2026 14:51
@greptile-apps
Copy link
Copy Markdown

greptile-apps Bot commented May 18, 2026 •
edited
Loading

Greptile Summary

This PR adds richer guard diagnostics to the dashboard while hiding local model details. The main changes are:

  • Redacts model metadata from dashboard event API responses.
  • Shows policy version, profile, rule pack, rule details, timestamp, and judge risk/latency in the inspector.
  • Removes visible judge model text and raw judge categories from dashboard diagnostics.
  • Updates dashboard event parsing, sample data, and built assets.

Confidence Score: 5/5

This looks safe to merge.

  • No blocking issues found in the changed code.

Important Files Changed

Filename Overview
internal/guard/app/server/server.go Adds response-only redaction for dashboard decision records.
web/guard-dashboard/src/dashboard/Inspector.tsx Expands clicked-event diagnostics while avoiding visible judge model/category details.
web/guard-dashboard/src/dashboard/api.ts Parses the new dashboard event metadata fields.

Reviews (2): Last reviewed commit: "feat(dashboard): show guard diagnostics" | Re-trigger Greptile

greptile-apps[bot]
greptile-apps Bot reviewed May 18, 2026
View reviewed changes
Comment thread internal/guard/app/server/server.go
Comment thread web/guard-dashboard/src/dashboard/Inspector.tsx
@hasandemirkiran hasandemirkiran mentioned this pull request May 18, 2026
Merged
3 tasks
@hasandemirkiran hasandemirkiran force-pushed the feat/eng-328-dashboard-diagnostics branch from b574bcd to b899b30 Compare May 18, 2026 15:03
@hasandemirkiran Graphite App
Copy link
Copy Markdown
Contributor Author

hasandemirkiran commented May 18, 2026

@greptileai

@hasandemirkiran hasandemirkiran force-pushed the feat/eng-328-dashboard-diagnostics branch from b899b30 to f968a5f Compare May 18, 2026 15:10
@hasandemirkiran hasandemirkiran force-pushed the feat/eng-325-remove-markov branch from 55dc4cc to 70cb383 Compare May 18, 2026 15:10
michiosw
michiosw previously approved these changes May 18, 2026
View reviewed changes
@michiosw Graphite App
Copy link
Copy Markdown
Contributor

michiosw commented May 18, 2026 •
edited by hasandemirkiran
Loading

Merge activity

  • May 18, 4:28 PM UTC: A user started a stack merge that includes this pull request via Graphite.
  • May 18, 4:34 PM UTC: Graphite couldn't merge this PR because it had merge conflicts.
  • May 18, 7:42 PM UTC: A user started a stack merge that includes this pull request via Graphite.
  • May 18, 7:43 PM UTC: @hasandemirkiran merged this pull request with Graphite.

@michiosw michiosw changed the base branch from feat/eng-325-remove-markov to graphite-base/156 May 18, 2026 16:29
@michiosw michiosw changed the base branch from graphite-base/156 to main May 18, 2026 16:32
@michiosw michiosw dismissed their stale review May 18, 2026 16:32

The base branch was changed.

@michiosw michiosw force-pushed the feat/eng-328-dashboard-diagnostics branch from f968a5f to d850ae3 Compare May 18, 2026 19:37
@hasandemirkiran hasandemirkiran force-pushed the feat/eng-328-dashboard-diagnostics branch from d850ae3 to 60aa70c Compare May 18, 2026 19:39
@hasandemirkiran hasandemirkiran mentioned this pull request May 18, 2026
Merged
5 tasks
@hasandemirkiran hasandemirkiran merged commit dcbdf4a into main May 18, 2026
5 checks passed
@hasandemirkiran hasandemirkiran deleted the feat/eng-328-dashboard-diagnostics branch May 18, 2026 19:43
@kontext-release-bot kontext-release-bot Bot mentioned this pull request May 18, 2026
Merged
michiosw pushed a commit that referenced this pull request May 18, 2026
@kontext-release-bot
chore(main): release 0.7.0 (#115)
3efdac7 
🤖 I have created a release *beep* *boop*
---


## [0.7.0](v0.6.0...v0.7.0) (2026-05-18)


### Features

* add local guard judge contract ([#132](#132)) ([ef6a3cd](ef6a3cd))
* **cli:** make kontext start local-first ([#140](#140)) ([02a34bf](02a34bf))
* **dashboard:** show guard diagnostics ([#156](#156)) ([dcbdf4a](dcbdf4a))
* **guard:** add deterministic policy engine ([#131](#131)) ([3a0f16e](3a0f16e))
* **guard:** add policy config store ([#135](#135)) ([520c3b6](520c3b6))
* **guard:** add policy profile dashboard ([#137](#137)) ([0b0e856](0b0e856))
* **guard:** connect deterministic policy and judge ([#154](#154)) ([25f0d32](25f0d32))
* manage local judge runtime ([#136](#136)) ([09f4fc0](09f4fc0))
* **runtime service:** Guard now starts a Unix-socket localruntime.Service alongside the existing HTTP daemon ([#122](#122)) ([3e87e12](3e87e12))
* **runtime service:** Introduce runtime service and make existing Unix socket more generic ([#121](#121)) ([2bbaa93](2bbaa93))


### Bug Fixes

* **dashboard:** contain command drawer text ([#157](#157)) ([4107e31](4107e31))
* **dashboard:** polish activity summary and log groups ([#153](#153)) ([fc79101](fc79101))
* harden npm dependency resolution ([#114](#114)) ([86eadf5](86eadf5))
* optimize judge fixture category matching ([#146](#146)) ([0bad7ab](0bad7ab))
* **repo:** remove repo-wide codeowners ([#133](#133)) ([73557f0](73557f0))

---
This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@greptile-apps greptile-apps[bot] greptile-apps[bot] left review comments

@michiosw michiosw michiosw left review comments

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@hasandemirkiran @michiosw