Skip to content

Add light mode to sentry-triage canvas (1.4.0) - #5705

Merged
aaronpowell merged 1 commit into
github:mainfrom
pierceboggan:pierceboggan-update-sentry-triage-canvas
Oct 27, 2026
Merged

aaronpowell merged 1 commit into
github:mainfrom
pierceboggan:pierceboggan-update-sentry-triage-canvas

Conversation

@pierceboggan

Copy link
Copy Markdown
Contributor

Pull Request Checklist

  • I have read and followed the CONTRIBUTING.md guidelines.
  • I have read and followed the Guidance for submissions involving paid services.
  • My contribution adds a new instruction, prompt, agent, skill, workflow, or canvas extension file in the correct directory.
  • The file follows the required naming convention.
  • The content is clearly structured and follows the example format.
  • I have tested my instructions, prompt, agent, skill, workflow, or canvas extension with GitHub Copilot.
  • I have run npm start and verified that README.md is up to date.
  • I am targeting the main branch for this pull request.

Description

Follow-up to #5611. Syncs the light mode commit (0095e23) by Natalie Harris from pierceboggan/sentry-canvas@natalierharris-polish-sentry-triage-canvas.

  • extensions/sentry-triage/styles.mjs: a dedicated light palette with Sentry purple accents, accessible status colors, and softer input shadows. The layout is unchanged.
  • extensions/sentry-triage/README.md: notes the light appearance.
  • plugins/sentry-triage/plugin.json: 1.3.0 → 1.4.0, with a "What's new in 1.4.0" note in the plugin README. marketplace.json regenerated.

Original plugin by Liz Tom (MIT).


Type of Contribution

  • New instruction file.
  • New prompt file.
  • New agent file.
  • New plugin.
  • New skill file.
  • New agentic workflow.
  • New canvas extension.
  • Update to existing instruction, prompt, agent, plugin, skill, workflow, or canvas extension.
  • Other (please specify):

Additional Notes

The source branch's package-lock.json is still left out (its URLs point at an internal feed).

Validation:

  • npm run plugin:validate: passed
  • npm run build: passed (only the sentry-triage version changed in marketplace.json)
  • node --check extensions/sentry-triage/styles.mjs: passed
  • No CRLF in the changed files
  • Not run: a manual visual check of the canvas in the app

By submitting this pull request, I confirm that my contribution abides by the Code of Conduct and will be licensed under the MIT License.

Sync the light palette from pierceboggan/sentry-canvas
(natalierharris-polish-sentry-triage-canvas, 0095e23): accessible
status colors, softer input shadows, and Sentry purple accents that
don't depend on the host app's color mode. Bump the plugin to 1.4.0.

Co-authored-by: Natalie Harris <natalierharris@users.noreply.github.com>
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
@pierceboggan
pierceboggan requested review from a team as code owners October 27, 2026 00:20
Copilot AI max review requested due to automatic review settings October 27, 2026 00:20
@github-actions github-actions Bot added canvas-extension PR touches canvas extensions plugin PR touches plugins labels Oct 27, 2026
@github-actions

Copy link
Copy Markdown
Contributor

🔒 PR Risk Scan Results

Scanned 5 changed file(s).

Severity Count
🔴 High 0
🟠 Medium 1
ℹ️ Info 0
Severity Rule File Line Match
🟠 package-exec-command extensions/sentry-triage/README.md 90 npx sentry auth login

This is an automated soft-gate report. Findings indicate review targets and do not block merge by themselves.

@github-actions

github-actions Bot commented Oct 27, 2026 •

Copy link
Copy Markdown
Contributor

🚦 Submission status: 👀 Ready for review

Risk tier: merge-risk:medium — New or substantially changed resource without privileged execution
Required to merge: passing submission-gate checks plus 1 approval from reviewers with write access.

Why this tier
  • Changes 46 lines (low-risk limit is 40)

Automated checks

Check Status Details
Line endings ✅ Passed Passed · logs
Spelling ✅ Passed Passed · logs
Generated README consistency ✅ Passed Passed · logs
Plugin and extension validation ✅ Passed Passed · logs
Canvas extension validation ✅ Passed Passed · logs
Plugin structure ✅ Passed Passed · logs
Risk scan ✅ Passed Passed · logs
Contributor reputation ✅ Passed Passed · logs
Duplicate resource scan ✅ Passed Passed · logs
PR quality signal ⏭️ Skipped Skipped by its workflow · logs
Canvas/plugin smoke test ✅ Passed Passed · logs

Review

  • Approvals: 0/1
  • Assigned reviewer: not assigned yet — comment /request-review to ask for one
  • Review target date: not set
  • Still needed: 1 more approval(s)
  • No staffed reviewer pool owns these files yet; any reviewer with write access counts as the domain reviewer.

Commands

Command Who What it does
/rerun-checks PR author, maintainers Re-runs failed or incomplete checks and re-evaluates this gate
/request-review PR author, maintainers Asks the review rotation to assign a reviewer (adds needs-reviewer)

Updated for e6b217c · gate run · This comment is maintained automatically — see submission gate docs.

@github-actions

Copy link
Copy Markdown
Contributor

🧩 Canvas smoke test

✅ Passed · 0 error(s), 0 warning(s)

Check Result
Syntax, imports, file safety ✅
Preview image (≥ 400×160 PNG) ✅
Plugin manifests ✅
Materialize ✅
Install (Copilot CLI) ✅

Plugin sentry-triage v1.4.0

Field Value
Description Scan live Sentry issues in a Copilot canvas, group them by urgency, and hand issues off for tracking or a fix PR.
Author Liz Tom — https://github.com/liztom
License MIT
Keywords canvas, copilot-canvas, copilot-extension, error-triage, incident-response, on-call, sentry
Extensions sentry-triage
Materialize ✅
Install ✅
Changed plugin files
  • plugins/sentry-triage/README.md
  • plugins/sentry-triage/plugin.json

Extension sentry-triage

sentry-triage preview

Field Value
Registered by sentry-triage
Package sentry-triage 1.2.0
Preview 544×306, 307.2 KB
Modules 12 (12 reachable from extension.mjs)
Files 18

Permissions and capabilities (static analysis of modules reachable from extension.mjs)

  • Spawns processes (node:child_process)
  • File system access (node:fs)
  • Network sockets / local server (node:http, node:net, ...)
  • Host information (node:os)
  • Outbound HTTP requests (fetch)
  • Reads environment variables (process.env)
  • Third-party runtime packages: sentry
Changed extension files
  • extensions/sentry-triage/README.md
  • extensions/sentry-triage/styles.mjs

Static checks compile modules without executing them. Minimum preview size is configurable with CANVAS_PREVIEW_MIN_WIDTH/CANVAS_PREVIEW_MIN_HEIGHT. Workflow run

@github-actions github-actions Bot added ready-for-review Submission passed intake validation and is ready for maintainer review and removed awaiting-automation labels Oct 27, 2026
@aaronpowell
aaronpowell merged commit 0dde2f8 into github:main Oct 27, 2026
27 of 28 checks passed

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🟡 Changes recommended

Several status-text colors fall below the 4.5:1 minimum contrast ratio on their tinted backgrounds.

1 open issue
What changed in this PR

Updates the Sentry Triage canvas to a host-independent light appearance, preserving its layout and triage workflow.

Changes:

  • Introduces light surfaces, revised status colors, and softer shadows.
  • Documents the appearance change and updates the plugin and marketplace version to 1.4.0.
File Description
plugins/​sentry-triage/​README.md Adds the 1.4.0 light-appearance release note.
plugins/​sentry-triage/​plugin.json Bumps the plugin version to 1.4.0.
extensions/​sentry-triage/​styles.mjs Defines the light palette and adjusts status colors and shadows.
extensions/​sentry-triage/​README.md Documents the host-independent light appearance.
.github/​plugin/​marketplace.json Updates the generated plugin version to 1.4.0.

Review effort: Max

Comment on lines +22 to +25
--st-danger: #c53036;
--st-warning: #946200;
--st-success: #187961;
--st-pink: #b52172;
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

canvas-extension PR touches canvas extensions merge-risk:medium plugin PR touches plugins ready-for-review Submission passed intake validation and is ready for maintainer review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants