fix: add eslint-config-prettier to shared config and SKIP_PACKAGES support

eslint-config-prettier was missing from the shared ESLint config, causing
unicorn/no-nested-ternary and unicorn/number-literal-case to conflict with
prettier formatting in scout-for-lol and other packages. Adding it as the
last config disables all conflicting rules.

Also adds SKIP_PACKAGES env var support to run-package-script.ts so the
Dagger CI can skip sjer.red (which needs Playwright for rehype-mermaid)
during the monorepo build step.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Author: claude

.dagger/src/index-infra.ts

@@ -70,7 +70,7 @@ export function getRustContainer(
       dag.cacheVolume(`apt-lib-rust-${RUST_VERSION}`),
     )
     .withExec(["apt-get", "update"])
-    .withExec(["apt-get", "install", "-y", "mold", "clang"])
+    .withExec(["apt-get", "install", "-y", "mold", "clang", "git"])
     .withMountedCache(
       "/usr/local/cargo/registry",
       dag.cacheVolume("cargo-registry"),

.dagger/src/index.ts

@@ -133,6 +133,8 @@ export class Monorepo {
         const c = container
           .withMountedDirectory("/workspace/packages/webring/dist", webringDist)
           .withWorkdir("/workspace")
+          // sjer.red needs Playwright for rehype-mermaid; validated separately in tier 0
+          .withEnvVariable("SKIP_PACKAGES", "sjer.red")
           .withExec(["bun", "run", "build"]);
         await c.sync();
         return c;

bun.lock

@@ -30,6 +30,7 @@
     "@eslint-community/eslint-plugin-eslint-comments": "^4.5.0",
     "@eslint/js": "^9.30.1",
     "@typescript-eslint/utils": "^8.46.0",
+    "eslint-config-prettier": "^10.1.8",
     "eslint-import-resolver-typescript-bun": "^0.0.104",
     "eslint-plugin-astro": "^1.5.0",
     "eslint-plugin-import": "^2.31.0",

packages/eslint-config/package.json

@@ -49,6 +49,7 @@ import { reactConfig } from "./configs/react.js";
 import { accessibilityConfig } from "./configs/accessibility.js";
 import { namingConfig } from "./configs/naming.js";
 import { customRulesPlugin } from "./rules/index.js";
+import prettierConfig from "eslint-config-prettier";
 
 export type RecommendedOptions = BaseConfigOptions &
   ImportsConfigOptions & {
@@ -146,35 +147,35 @@ export function recommended(
     },
   };
 
-  if (customRules.reactRules && react) {
+  if (customRules.reactRules === true && react) {
     customRulesConfig.rules = {
       ...customRulesConfig.rules,
       "custom-rules/no-use-effect": "warn",
     };
   }
 
-  if (customRules.noDtoNaming) {
+  if (customRules.noDtoNaming === true) {
     customRulesConfig.rules = {
       ...customRulesConfig.rules,
       "custom-rules/no-dto-naming": "error",
     };
   }
 
-  if (customRules.structuredLogging) {
+  if (customRules.structuredLogging === true) {
     customRulesConfig.rules = {
       ...customRulesConfig.rules,
       "custom-rules/prefer-structured-logging": "error",
     };
   }
 
-  if (customRules.noShadcnThemeTokens) {
+  if (customRules.noShadcnThemeTokens === true) {
     customRulesConfig.rules = {
       ...customRulesConfig.rules,
       "custom-rules/no-shadcn-theme-tokens": "error",
     };
   }
 
-  if (customRules.analysisRules) {
+  if (customRules.analysisRules === true) {
     customRulesConfig.rules = {
       ...customRulesConfig.rules,
       "custom-rules/knip-unused": "warn",
@@ -244,6 +245,9 @@ export function recommended(
     },
   });
 
+  // Must be LAST: disables ESLint rules that conflict with prettier formatting
+  configs.push(prettierConfig);
+
   return configs;
 }
 

packages/eslint-config/src/index.ts

@@ -65,11 +65,17 @@ async function main(): Promise<void> {
     process.exit(1);
   }
 
+  const skipPackages = new Set(
+    (process.env["SKIP_PACKAGES"] ?? "").split(",").filter(Boolean),
+  );
+
   const packageJsonPaths = await getPackageJsonPaths();
   const metas = await Promise.all(
     packageJsonPaths.map((path) => loadPackageMeta(path)),
   );
-  const runnable = metas.filter((meta) => meta.scripts[scriptName]);
+  const runnable = metas
+    .filter((meta) => meta.scripts[scriptName])
+    .filter((meta) => !skipPackages.has(meta.name));
   const skipped = metas.length - runnable.length;
   const failures: Array<{ dir: string; error: unknown }> = [];