From d860c328be6cbe4fa49d3141c7816463bfdc6862 Mon Sep 17 00:00:00 2001 From: murashit Date: Sat, 27 Jun 2026 23:20:08 +0900 Subject: [PATCH] Simplify import boundary Grit references --- .../no-app-server-connection-boundary-imports.grit | 2 -- .../no-app-server-protocol-boundary-imports.grit | 2 -- .../architecture/no-app-server-root-module-imports.grit | 2 -- .../architecture/no-app-server-subfolder-root-imports.grit | 2 -- .../no-chat-app-server-outer-layer-imports.grit | 2 -- .../no-chat-application-outer-layer-imports.grit | 2 -- .../architecture/no-chat-host-rendering-layer-imports.grit | 2 -- .../no-chat-panel-runtime-boundary-imports.grit | 2 -- .../no-chat-presentation-outer-layer-imports.grit | 2 -- .../grit/architecture/no-chat-ui-outer-layer-imports.grit | 2 -- .../architecture/no-chat-workspace-boundary-imports.grit | 2 -- .../grit/architecture/no-domain-outer-layer-imports.grit | 2 -- .../architecture/no-feature-workspace-boundary-imports.grit | 2 -- .../no-generated-app-server-boundary-imports.grit | 2 -- .../grit/architecture/no-lower-level-feature-imports.grit | 2 -- scripts/grit/architecture/no-preact-signal-imports.grit | 2 -- .../no-settings-app-server-boundary-imports.grit | 2 -- .../architecture/no-workspace-chat-internal-imports.grit | 2 -- scripts/grit/dom/no-dom-events-imports.grit | 2 -- scripts/grit/dom/no-ui-root-imports.grit | 2 -- tests/scripts/grit-policy.test.mjs | 6 ------ 21 files changed, 46 deletions(-) diff --git a/scripts/grit/architecture/no-app-server-connection-boundary-imports.grit b/scripts/grit/architecture/no-app-server-connection-boundary-imports.grit index 58fbd6d9..39dda4ad 100644 --- a/scripts/grit/architecture/no-app-server-connection-boundary-imports.grit +++ b/scripts/grit/architecture/no-app-server-connection-boundary-imports.grit @@ -3,8 +3,6 @@ language js private pattern js_module_reference() { or { JsImport(), - JsExportNamedFromClause(), - JsExportFromClause(), JsImportCallExpression() } } diff --git a/scripts/grit/architecture/no-app-server-protocol-boundary-imports.grit b/scripts/grit/architecture/no-app-server-protocol-boundary-imports.grit index 45da2921..8dae2737 100644 --- a/scripts/grit/architecture/no-app-server-protocol-boundary-imports.grit +++ b/scripts/grit/architecture/no-app-server-protocol-boundary-imports.grit @@ -3,8 +3,6 @@ language js private pattern js_module_reference() { or { JsImport(), - JsExportNamedFromClause(), - JsExportFromClause(), JsImportCallExpression() } } diff --git a/scripts/grit/architecture/no-app-server-root-module-imports.grit b/scripts/grit/architecture/no-app-server-root-module-imports.grit index f4ec3c7d..dcfad34b 100644 --- a/scripts/grit/architecture/no-app-server-root-module-imports.grit +++ b/scripts/grit/architecture/no-app-server-root-module-imports.grit @@ -3,8 +3,6 @@ language js private pattern js_module_reference() { or { JsImport(), - JsExportNamedFromClause(), - JsExportFromClause(), JsImportCallExpression() } } diff --git a/scripts/grit/architecture/no-app-server-subfolder-root-imports.grit b/scripts/grit/architecture/no-app-server-subfolder-root-imports.grit index db0e9921..aa7f7df6 100644 --- a/scripts/grit/architecture/no-app-server-subfolder-root-imports.grit +++ b/scripts/grit/architecture/no-app-server-subfolder-root-imports.grit @@ -3,8 +3,6 @@ language js private pattern js_module_reference() { or { JsImport(), - JsExportNamedFromClause(), - JsExportFromClause(), JsImportCallExpression() } } diff --git a/scripts/grit/architecture/no-chat-app-server-outer-layer-imports.grit b/scripts/grit/architecture/no-chat-app-server-outer-layer-imports.grit index 73fb5d89..430b5270 100644 --- a/scripts/grit/architecture/no-chat-app-server-outer-layer-imports.grit +++ b/scripts/grit/architecture/no-chat-app-server-outer-layer-imports.grit @@ -3,8 +3,6 @@ language js private pattern js_module_reference() { or { JsImport(), - JsExportNamedFromClause(), - JsExportFromClause(), JsImportCallExpression() } } diff --git a/scripts/grit/architecture/no-chat-application-outer-layer-imports.grit b/scripts/grit/architecture/no-chat-application-outer-layer-imports.grit index 6e4dca6d..c366b2a5 100644 --- a/scripts/grit/architecture/no-chat-application-outer-layer-imports.grit +++ b/scripts/grit/architecture/no-chat-application-outer-layer-imports.grit @@ -3,8 +3,6 @@ language js private pattern js_module_reference() { or { JsImport(), - JsExportNamedFromClause(), - JsExportFromClause(), JsImportCallExpression() } } diff --git a/scripts/grit/architecture/no-chat-host-rendering-layer-imports.grit b/scripts/grit/architecture/no-chat-host-rendering-layer-imports.grit index 3363ae9c..c6e26fd7 100644 --- a/scripts/grit/architecture/no-chat-host-rendering-layer-imports.grit +++ b/scripts/grit/architecture/no-chat-host-rendering-layer-imports.grit @@ -3,8 +3,6 @@ language js private pattern js_module_reference() { or { JsImport(), - JsExportNamedFromClause(), - JsExportFromClause(), JsImportCallExpression() } } diff --git a/scripts/grit/architecture/no-chat-panel-runtime-boundary-imports.grit b/scripts/grit/architecture/no-chat-panel-runtime-boundary-imports.grit index b9886462..4cd00b86 100644 --- a/scripts/grit/architecture/no-chat-panel-runtime-boundary-imports.grit +++ b/scripts/grit/architecture/no-chat-panel-runtime-boundary-imports.grit @@ -3,8 +3,6 @@ language js private pattern js_module_reference() { or { JsImport(), - JsExportNamedFromClause(), - JsExportFromClause(), JsImportCallExpression() } } diff --git a/scripts/grit/architecture/no-chat-presentation-outer-layer-imports.grit b/scripts/grit/architecture/no-chat-presentation-outer-layer-imports.grit index 97f0813d..1a53ee8c 100644 --- a/scripts/grit/architecture/no-chat-presentation-outer-layer-imports.grit +++ b/scripts/grit/architecture/no-chat-presentation-outer-layer-imports.grit @@ -3,8 +3,6 @@ language js private pattern js_module_reference() { or { JsImport(), - JsExportNamedFromClause(), - JsExportFromClause(), JsImportCallExpression() } } diff --git a/scripts/grit/architecture/no-chat-ui-outer-layer-imports.grit b/scripts/grit/architecture/no-chat-ui-outer-layer-imports.grit index 4e253380..bb80fcdb 100644 --- a/scripts/grit/architecture/no-chat-ui-outer-layer-imports.grit +++ b/scripts/grit/architecture/no-chat-ui-outer-layer-imports.grit @@ -3,8 +3,6 @@ language js private pattern js_module_reference() { or { JsImport(), - JsExportNamedFromClause(), - JsExportFromClause(), JsImportCallExpression() } } diff --git a/scripts/grit/architecture/no-chat-workspace-boundary-imports.grit b/scripts/grit/architecture/no-chat-workspace-boundary-imports.grit index c3028bb9..c157427a 100644 --- a/scripts/grit/architecture/no-chat-workspace-boundary-imports.grit +++ b/scripts/grit/architecture/no-chat-workspace-boundary-imports.grit @@ -3,8 +3,6 @@ language js private pattern js_module_reference() { or { JsImport(), - JsExportNamedFromClause(), - JsExportFromClause(), JsImportCallExpression() } } diff --git a/scripts/grit/architecture/no-domain-outer-layer-imports.grit b/scripts/grit/architecture/no-domain-outer-layer-imports.grit index c630498a..0996358e 100644 --- a/scripts/grit/architecture/no-domain-outer-layer-imports.grit +++ b/scripts/grit/architecture/no-domain-outer-layer-imports.grit @@ -3,8 +3,6 @@ language js private pattern js_module_reference() { or { JsImport(), - JsExportNamedFromClause(), - JsExportFromClause(), JsImportCallExpression() } } diff --git a/scripts/grit/architecture/no-feature-workspace-boundary-imports.grit b/scripts/grit/architecture/no-feature-workspace-boundary-imports.grit index a48952f6..84d862d8 100644 --- a/scripts/grit/architecture/no-feature-workspace-boundary-imports.grit +++ b/scripts/grit/architecture/no-feature-workspace-boundary-imports.grit @@ -3,8 +3,6 @@ language js private pattern js_module_reference() { or { JsImport(), - JsExportNamedFromClause(), - JsExportFromClause(), JsImportCallExpression() } } diff --git a/scripts/grit/architecture/no-generated-app-server-boundary-imports.grit b/scripts/grit/architecture/no-generated-app-server-boundary-imports.grit index 8736b9af..ed3796c2 100644 --- a/scripts/grit/architecture/no-generated-app-server-boundary-imports.grit +++ b/scripts/grit/architecture/no-generated-app-server-boundary-imports.grit @@ -3,8 +3,6 @@ language js private pattern js_module_reference() { or { JsImport(), - JsExportNamedFromClause(), - JsExportFromClause(), JsImportCallExpression() } } diff --git a/scripts/grit/architecture/no-lower-level-feature-imports.grit b/scripts/grit/architecture/no-lower-level-feature-imports.grit index 1bed0885..13094956 100644 --- a/scripts/grit/architecture/no-lower-level-feature-imports.grit +++ b/scripts/grit/architecture/no-lower-level-feature-imports.grit @@ -3,8 +3,6 @@ language js private pattern js_module_reference() { or { JsImport(), - JsExportNamedFromClause(), - JsExportFromClause(), JsImportCallExpression() } } diff --git a/scripts/grit/architecture/no-preact-signal-imports.grit b/scripts/grit/architecture/no-preact-signal-imports.grit index 446611ef..7eb6386d 100644 --- a/scripts/grit/architecture/no-preact-signal-imports.grit +++ b/scripts/grit/architecture/no-preact-signal-imports.grit @@ -3,8 +3,6 @@ language js private pattern js_module_reference() { or { JsImport(), - JsExportNamedFromClause(), - JsExportFromClause(), JsImportCallExpression() } } diff --git a/scripts/grit/architecture/no-settings-app-server-boundary-imports.grit b/scripts/grit/architecture/no-settings-app-server-boundary-imports.grit index db84657f..0537e57d 100644 --- a/scripts/grit/architecture/no-settings-app-server-boundary-imports.grit +++ b/scripts/grit/architecture/no-settings-app-server-boundary-imports.grit @@ -3,8 +3,6 @@ language js private pattern js_module_reference() { or { JsImport(), - JsExportNamedFromClause(), - JsExportFromClause(), JsImportCallExpression() } } diff --git a/scripts/grit/architecture/no-workspace-chat-internal-imports.grit b/scripts/grit/architecture/no-workspace-chat-internal-imports.grit index c78575a8..fef4901c 100644 --- a/scripts/grit/architecture/no-workspace-chat-internal-imports.grit +++ b/scripts/grit/architecture/no-workspace-chat-internal-imports.grit @@ -3,8 +3,6 @@ language js private pattern js_module_reference() { or { JsImport(), - JsExportNamedFromClause(), - JsExportFromClause(), JsImportCallExpression() } } diff --git a/scripts/grit/dom/no-dom-events-imports.grit b/scripts/grit/dom/no-dom-events-imports.grit index 61d4c7cb..76482649 100644 --- a/scripts/grit/dom/no-dom-events-imports.grit +++ b/scripts/grit/dom/no-dom-events-imports.grit @@ -3,8 +3,6 @@ language js private pattern js_module_reference() { or { JsImport(), - JsExportNamedFromClause(), - JsExportFromClause(), JsImportCallExpression() } } diff --git a/scripts/grit/dom/no-ui-root-imports.grit b/scripts/grit/dom/no-ui-root-imports.grit index 1410794d..c30a4f12 100644 --- a/scripts/grit/dom/no-ui-root-imports.grit +++ b/scripts/grit/dom/no-ui-root-imports.grit @@ -3,8 +3,6 @@ language js private pattern js_module_reference() { or { JsImport(), - JsExportNamedFromClause(), - JsExportFromClause(), JsImportCallExpression() } } diff --git a/tests/scripts/grit-policy.test.mjs b/tests/scripts/grit-policy.test.mjs index 8e6f59d9..8c33c4b9 100644 --- a/tests/scripts/grit-policy.test.mjs +++ b/tests/scripts/grit-policy.test.mjs @@ -203,7 +203,6 @@ export const status = signal("idle"); path.join(cwd, "src/shared/ui/signal-escapes.tsx"), ` import type { Signal } from "@preact/signals"; -export { signal } from "@preact/signals"; export type SignalValue = Signal; @@ -343,7 +342,6 @@ export const render = renderUiRoot; path.join(cwd, "src/features/chat/ui/root-escapes.tsx"), ` import type { RootRenderer } from "../../../shared/ui/ui-root.dom"; -export { renderUiRoot } from "../../../shared/ui/ui-root.dom"; export type { RootRenderer }; @@ -398,7 +396,6 @@ export function timestamp(): number { "Do not import @preact/signals from this module.", "Do not import @preact/signals from this module.", "Do not import @preact/signals from this module.", - "Do not import @preact/signals from this module.", ]); expect(pluginMessages(report, "src/features/chat/ui/dom-bridge-escape.tsx")).toEqual( Array.from({ length: 37 }, () => DOM_BOUNDARY_MESSAGE), @@ -414,7 +411,6 @@ export function timestamp(): number { "Import the Preact root adapter only from explicit root bridge files.", "Import the Preact root adapter only from explicit root bridge files.", "Import the Preact root adapter only from explicit root bridge files.", - "Import the Preact root adapter only from explicit root bridge files.", ]); expect(pluginMessages(report, "src/features/chat/application/state/message-stream.ts")).toEqual([ "Keep this state module deterministic and free of app-server, Obsidian, scheduling, and browser side effects.", @@ -812,7 +808,6 @@ export const value = statusText; "Domain modules must stay pure; outer layers may depend on domain, not the reverse.", "Domain modules must stay pure; outer layers may depend on domain, not the reverse.", "Domain modules must stay pure; outer layers may depend on domain, not the reverse.", - "Domain modules must stay pure; outer layers may depend on domain, not the reverse.", ]); }); @@ -1262,7 +1257,6 @@ export type Item = MessageStreamItem; path.join(cwd, "src/features/chat/domain/message-stream/outer-shapes.ts"), ` import type { ChatStateStore } from "../../application/state/store"; -export { createChatStateStore } from "../../application/state/store"; export type OuterStore = ChatStateStore;