feat(import): scale GitHub imports and add secure private access - #233
Draft
ThisIs-Developer wants to merge 6 commits into
Draft
feat(import): scale GitHub imports and add secure private access#233ThisIs-Developer wants to merge 6 commits into
ThisIs-Developer wants to merge 6 commits into
Conversation
Deploying markdownviewer with
|
| Latest commit: |
4a8bc33
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://63c23024.markdownviewer.pages.dev |
| Branch Preview URL: | https://agent-show-all-github-markdo.markdownviewer.pages.dev |
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
️✅ There are no secrets present in this pull request anymore.If these secrets were true positive and are still valid, we highly recommend you to revoke them. 🦉 GitGuardian detects secrets in your source code to help developers and security teams secure the modern development process. You are seeing this because you or someone else with access to this repository has authorized GitGuardian to scan your pull request. |
ThisIs-Developer
force-pushed
the
agent/show-all-github-markdown-files
branch
from
July 30, 2026 13:26
fd41696 to
fb0afce
Compare
ThisIs-Developer
force-pushed
the
agent/show-all-github-markdown-files
branch
from
July 30, 2026 13:29
fb0afce to
eba027d
Compare
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.
What changed
/; every import is pinned to the resolved immutable commit SHA.Contents: Read-only.repo-scope warning.https://api.github.com.Select Markdown files to importpicker at 760px, with file search, selected/total counts, icon-based select all/deselect all, and collapsible GitHub-style rows.Why
Large repositories were capped at the first 30 Markdown files. The importer assumed the first URL path segment was the complete ref, private repositories had no supported access path, bulk imports flooded the tab bar, and closing hundreds of tabs repeatedly rebuilt the UI. The first PAT UI also presented only fine-grained tokens and allowed its expanded card to shrink inside short viewports, clipping controls above the footer.
Security model
A browser-only application cannot promise zero compromise risk. This implementation minimizes exposure: PATs are never placed in URLs, logs,
localStorage, the Markdown Viewer backend, or requests toraw.githubusercontent.com; authenticated fetches reject non-api.github.comdestinations and cross-origin redirects. Fine-grained access remains the recommended default because it has a smaller blast radius. Classic PATs are clearly identified as broad-access compatibility credentials. Session-only access remains the safest retention choice.References:
User impact
Validation
node --check script.jsnode --check desktop-app/resources/js/script.jsgit diff --checkapi.github.comContents requests and never the raw host;Known unrelated baseline check
npm run buildstops because the repository baseline references the absentfunctions/api/report-issue.jsfile intests/helpers/static-build-check.mjs.