Skip to content

feat(web,macos): surface stale_classification_reasons in profile editors - #1509

Merged
Dumbris merged 1 commit into
mainfrom
fix/issues-b12-frontend-macos-tray
Oct 5, 2026
Merged

Dumbris merged 1 commit into
mainfrom
fix/issues-b12-frontend-macos-tray

Conversation

@Dumbris

@Dumbris Dumbris commented Oct 5, 2026 •

Copy link
Copy Markdown
Member

Summary

Surface stale classification reasons in profile editors across web and macOS interfaces, allowing users to understand why tools are marked as stale when editing tool policies.

Items Fixed

  • Follow-ups from post-merge Sol 6.1 review of Spec 108/109 PRs #1451: Surface stale_classification_reasons in profile editors
    • Regenerated contracts.ts to include stale_classification_reasons field
    • Updated ProfileToolTable component to display orphan stale classification from server responses (neutral fallback for older daemons)
    • Updated macOS ProfileEditor to show stale classification reasons alongside tool status
    • Tests: Frontend unit tests in profile-stale-reasons.spec.ts (2 tests), macOS unit tests in ProfileEditorModelTests.swift (1 test)

Review Status

Clean after 1 round(s).

Closes

Refs #1451

…ors (refs #1451)

Regenerate contracts.ts with the field; ProfileToolTable and the macOS
ProfileEditor word each orphan stale classification from the server's
reason (CLI wording), neutral for older daemons. Parity row 2 cites the
new web/macOS tests.
@Dumbris
Dumbris enabled auto-merge (squash) October 5, 2026 13:01
@cloudflare-workers-and-pages

Copy link
Copy Markdown

Deploying mcpproxy-docs with  Cloudflare Pages  Cloudflare Pages

Latest commit: 107411c
Status: ✅  Deploy successful!
Preview URL: https://8e22052e.mcpproxy-docs.pages.dev
Branch Preview URL: https://fix-issues-b12-frontend-maco.mcpproxy-docs.pages.dev

View logs

@codecov-commenter

Copy link
Copy Markdown

⚠️ Please install the 'codecov app svg image' to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@Dumbris
Dumbris merged commit 4623cf0 into main Oct 5, 2026
44 checks passed
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