Skip to content

chore(lint): enforce renderer process boundary - #3484

Merged
setchy merged 1 commit into
mainfrom
chore/lint-renderer-process-boundary
Oct 24, 2026
Merged

setchy merged 1 commit into
mainfrom
chore/lint-renderer-process-boundary

Conversation

@setchy

@setchy setchy commented Oct 24, 2026

Copy link
Copy Markdown
Member

Summary

ARCHITECTURE.md says the renderer never reaches into the main process directly; it goes through window.gitify. This enforces that for imports.

  • Adds a shared processBoundaryPattern built with rendererBoundaryPattern / forbiddenLayerGlobs.
  • Adds a renderer-wide override for routes, App.tsx, index.tsx, and test helpers.
  • Composes the same pattern into every layer override, because a more specific no-restricted-imports override replaces the parent rule rather than merging with it.
  • Type-only imports remain allowed (allowTypeImports: true).
  • Tests and test helpers are excluded, consistent with the other boundary rules.

Verification

  • Confirmed a renderer route importing src/main or src/preload is reported.
  • Confirmed a component importing src/main is reported while the existing component rules still apply.
  • pnpm check passes.

@setchy
setchy requested a review from afonsojramos as a code owner October 24, 2026 13:14
@github-actions github-actions Bot added the chore Miscellaneous tasks label Oct 24, 2026
@sonarqubecloud

Copy link
Copy Markdown

@setchy
setchy merged commit 0cb47e6 into main Oct 24, 2026
19 checks passed
@setchy
setchy deleted the chore/lint-renderer-process-boundary branch October 24, 2026 13:26
@github-actions github-actions Bot added this to the Release 7.11.0 milestone Oct 24, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

chore Miscellaneous tasks

Development

Successfully merging this pull request may close these issues.

1 participant