Skip to content

fix(web): wrap Explorer with GraphProvider - #540

Merged
GSF-001 merged 1 commit into
ARCLUX.mainfrom
fix/explorer-missing-graphprovider
Aug 22, 2026
Merged

fix(web): wrap Explorer with GraphProvider#540
GSF-001 merged 1 commit into
ARCLUX.mainfrom
fix/explorer-missing-graphprovider

Conversation

@GSF-001

@GSF-001 GSF-001 commented Aug 22, 2026

Copy link
Copy Markdown
Owner

DependencyList (and other graph-context consumers rendered inside Explorer) call useGraphContext(), but Explorer never rendered a wrapper — the comment at line 52 incorrectly assumed one was already present higher in the tree. This crashed the Dependencies tab with 'useGraphContext must be used within a ' as soon as the web dashboard was actually run. Fixed by wrapping Explorer's return JSX with .

What changed

How was this verified

Checklist

  • npx tsc --noEmit -p apps/web/tsconfig.json (if touching apps/web)
  • Tested against at least 1 fixture in playground/ (if touching parser/detector/pipeline)
  • PROGRES.md updated if this changes the status of a previously empty/stub file
  • Doesn't duplicate existing file/logic (checked first with grep/cat)
  • Updated relevant progres/PROGRES-*.md file with a dated entry (## YYYY-MM-DD — title)
  • Ran scripts/log-progress.sh instead of hand-editing PROGRES files, where applicable
  • Tested on Termux (or noted why not applicable)
  • No empty files introduced (check via the empty-file scan in PROGRES.md)

DependencyList (and other graph-context consumers rendered inside
Explorer) call useGraphContext(), but Explorer never rendered a
<GraphProvider> wrapper — the comment at line 52 incorrectly assumed
one was already present higher in the tree. This crashed the
Dependencies tab with 'useGraphContext must be used within a
<GraphProvider>' as soon as the web dashboard was actually run.
Fixed by wrapping Explorer's return JSX with <GraphProvider
repoUrl={repoUrl} branch={branch}>.
@GSF-001
GSF-001 requested a review from ManSio as a code owner August 22, 2026 11:25
@GSF-001
GSF-001 merged commit c6e57ef into ARCLUX.main Aug 22, 2026
4 checks passed
@GSF-001
GSF-001 deleted the fix/explorer-missing-graphprovider branch August 22, 2026 11:25
@github-actions

Copy link
Copy Markdown

ThreatCrush Security Scan

11 finding(s)

MEDIUM: 4 | LOW: 7

Severity Rule Location
MEDIUM js-dynamic-code-execution packages/security-analysis/source/UnsafePatternDetector.ts:58
MEDIUM js-dynamic-code-execution packages/security-analysis/source/UnsafePatternDetector.ts:61
MEDIUM js-dynamic-code-execution packages/security-analysis/source/UnsafePatternDetector.ts:62
MEDIUM js-dynamic-code-execution packages/security-analysis/source/UnsafePatternDetector.ts:73
LOW secret-aws-access-key tests/boundaries.test.ts:139
LOW secret-jwt tests/boundaries.test.ts:140
LOW secret-private-key tests/boundaries.test.ts:141
LOW secret-openai-key tests/fixtures/security-leaks/app.ts:6
LOW secret-generic-api-key tests/fixtures/security-leaks/app.ts:6
LOW secret-generic-credential tests/fixtures/security-leaks/app.ts:9
LOW secret-generic-credential tests/fixtures/security-leaks/app.ts:16

Snippets are redacted; ThreatCrush never prints matched credential material.

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.

1 participant