Skip to content

Link quarantine refusals to authorized review history - #1085

Open
time-attack wants to merge 1 commit into
yc-software:mainfrom
time-attack:fix/quarantine-review-handoff
Open

time-attack wants to merge 1 commit into
yc-software:mainfrom
time-attack:fix/quarantine-review-handoff

Conversation

@time-attack

@time-attack time-attack commented Sep 10, 2026 •

Copy link
Copy Markdown
Collaborator

Summary

  • link quarantine refusals to authorized session history across direct Slack, web, approval-resume, and addressed async-recovery paths
  • canonicalize and validate HTTP(S) review URLs, resolve legacy recovery thread references, and render safe web links without exposing internal screening reasons
  • accurately describe Auto, Strict, Dangerous, and command-policy behavior

Security enforcement is unchanged; this only corrects user-facing copy, review handoff wiring, safe rendering, and posture documentation.

Fixes #574, reported by @ianTPE.

Verification

  • 104 affected root tests passed
  • 30 affected web tests passed
  • root, contract, and web typechecks passed
  • ESLint, oxlint, Prettier, and diff checks passed
  • independently reviewed and approved after two review passes

Real Slack QA was not performed because the full dev instance was blocked by unavailable local Docker/Sprites infrastructure. A synthetic browser demo using the branch's actual shared formatters, resource registry, and safe Lit link renderer was completed instead; no production data or credentials were used.


View with [code]smith Autofix with [code]smith
Need help on this PR? Tag @codesmith-bot with what you need. Autofix is disabled.

@time-attack

Copy link
Copy Markdown
Collaborator Author

Rendered quarantine-review handoff

This uses the branch's actual shared formatters and safe web link renderer with synthetic local data. It is a rendering/wiring demonstration, not a screenshot of a real Slack workspace. Direct refusal, addressed recovery, stored web refusal, and the no-admin-URL fallback are shown.
quarantine-review-handoff

@time-attack
time-attack force-pushed the fix/quarantine-review-handoff branch from bae24cb to 8db46e1 Compare September 11, 2026 00:02
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.

Quarantine refusal points at a review flow that isn't reachable; tightening posture to strict silently removes all content screening

1 participant