feat(extension): add public-safe packet copy and private blocker actions in GitHub overlay - #202
Merged
JSONbored merged 11 commits intoJun 2, 2026
Conversation
JSONbored
previously requested changes
Jun 1, 2026
JSONbored
left a comment
Owner
There was a problem hiding this comment.
@jonathanchang31 failing CI, please resolve.
jonathanchang31
force-pushed
the
feat/public-safe-packet-private-blocker-actions
branch
from
June 1, 2026 20:33
6cd1bd9 to
e66fec5
Compare
Signed-off-by: Jonathan Chang <55106972+jonathanchang31@users.noreply.github.com>
Contributor
Author
|
@JSONbored Could you plz review my PR again? Thanks! && I Love Gittensory! |
JSONbored
dismissed
their stale review
June 2, 2026 04:41
@jonathanchang31 current head looks ready from this pass.
A few notes:
- The extension-scoped route boundary is preserved: private pull context stays behind the extension session path.
- The public-safe packet path fails closed instead of trying to partially redact unsafe content.
- I checked the current fetched head and did not find the conflict markers called out by the active scanner thread, so that thread looks stale.
- No code changes requested.
Validation expected:
- Current GitHub checks are green.
- Maintainer should resolve the stale review/thread state before landing this.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
This change implements issue #147 by extending the browser extension and API to support:
The implementation reuses existing public-safety sanitization behavior and keeps extension output within public/private boundaries.
Related issue
Closes: #147
Change Type
Real Behavior Proof
1) Extension context now includes action payloads
GET /v1/extension/pull-context?...returns:actions[]containing:id: "copy_public_safe_packet"id: "view_private_blockers"2) Public-safe packet output is sanitized
Integration checks confirm packet content excludes sensitive terms such as:
3) Private blockers remain auth-scoped
Extension context remains accessible only through extension-scoped session flow; private blocker details are surfaced only in that authenticated context.
4) Extension UI behavior
Overlay now renders:
Checklist