@hublo/sentinel 1.2.0-alpha.0 → 1.2.0-alpha.2

package/dist/index.js

@@ -7,8 +7,7 @@ import {
   registerAdapters,
   resolve,
   setDefaultRunner
-} from "./chunk-TWL6T237.js";
-import "./chunk-XSCDJZDY.js";
+} from "./chunk-SA2RMTVI.js";
 export {
   BaseAdapter,
   all,

package/dist/roles/build/toolchain.d.ts

@@ -0,0 +1,5 @@
+export { default as tailwindcss } from '@tailwindcss/vite';
+export { default as react } from '@vitejs/plugin-react';
+export { nitro } from 'nitro/vite';
+export { default as svgr } from 'vite-plugin-svgr';
+export { Plugin, PluginOption, SassPreprocessorOptions, UserConfig, UserConfigExport, defineConfig, loadEnv, mergeConfig } from 'vite';

package/dist/roles/build/toolchain.js

@@ -0,0 +1,16 @@
+// src/roles/build/toolchain.ts
+import { default as default2 } from "@tailwindcss/vite";
+import { default as default3 } from "@vitejs/plugin-react";
+import { nitro } from "nitro/vite";
+import { default as default4 } from "vite-plugin-svgr";
+import { defineConfig, loadEnv, mergeConfig } from "vite";
+export {
+  defineConfig,
+  loadEnv,
+  mergeConfig,
+  nitro,
+  default3 as react,
+  default4 as svgr,
+  default2 as tailwindcss
+};
+//# sourceMappingURL=toolchain.js.map

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@hublo/sentinel",
-  "version": "1.2.0-alpha.0",
+  "version": "1.2.0-alpha.2",
   "description": "One CLI that guards code health across Hublo repos: shared lint/typescript/build/test presets, static & dynamic analysis, and architecture checks.",
   "type": "module",
   "license": "MIT",
@@ -39,8 +39,8 @@
       "default": "./dist/tsconfig/node.json"
     },
     "./build/react": {
-      "types": "./dist/roles/build/react-app.d.ts",
-      "import": "./dist/roles/build/react-app.js"
+      "types": "./dist/roles/build/toolchain.d.ts",
+      "import": "./dist/roles/build/toolchain.js"
     }
   },
   "files": [
@@ -62,6 +62,7 @@
     "eslint-plugin-styled-components-a11y": "2.2.0",
     "jsonc-parser": "^3.3.1",
     "nitro": "3.0.260415-beta",
+    "oxc-parser": "0.147.0",
     "oxfmt": "0.63.0",
     "oxlint": "1.77.0",
     "oxlint-tsgolint": "7.0.2001",

package/dist/chunk-XSCDJZDY.js

@@ -1,26 +0,0 @@
-// src/roles/build/presets/base.json
-var base_default = {
-  target: "chrome89",
-  allowedHosts: [".hubpreprod.com", ".playground-hublo.com"],
-  publicAssets: [
-    {
-      dir: "public/branding",
-      baseURL: "/branding",
-      maxAge: 31536e3
-    }
-  ]
-};
-
-// src/roles/build/preset-data.ts
-var REACT_APP_DEFAULTS = base_default;
-
-// src/shared/deep-merge.ts
-function isPlainObject(value) {
-  return typeof value === "object" && value !== null && !Array.isArray(value);
-}
-
-export {
-  REACT_APP_DEFAULTS,
-  isPlainObject
-};
-//# sourceMappingURL=chunk-XSCDJZDY.js.map

package/dist/roles/build/react-app.d.ts

@@ -1,116 +0,0 @@
-import { PluginOption, UserConfig } from 'vite';
-export { defineConfig, loadEnv } from 'vite';
-
-/** The convention values, in one place so `--inspect` can report what an app departs from. */
-declare const REACT_APP_DEFAULTS: {
-    target: string;
-    allowedHosts: string[];
-    publicAssets: {
-        dir: string;
-        baseURL: string;
-        maxAge: number;
-    }[];
-};
-
-/** Where the router's routes live and where it writes its generated tree. */
-interface RouterPaths {
-    routesDirectory: string;
-    generatedRouteTree: string;
-    /** Extra lines appended to the generated tree. Two of the three apps use it. */
-    routeTreeFileFooter?: string[];
-}
-/**
- * The TanStack factories, supplied by the app.
- *
- * Typed loosely on purpose: sentinel decides WHEN to call them and with what, and has no
- * business asserting their return shape. Narrowing this would only couple sentinel to a
- * version of a package it does not own.
- */
-interface TanstackFactories {
-    start: (options: {
-        router: RouterPaths;
-    }) => PluginOption[];
-    router: (options: RouterPaths & {
-        target: string;
-        autoCodeSplitting: boolean;
-    }) => PluginOption;
-}
-/** Everything an app declares. */
-interface ReactAppOptions {
-    /** The app's own directory. Always `__dirname`. */
-    root: string;
-    /** Vite's `mode`, from the config callback. */
-    mode: string;
-    /** Served under, e.g. `/console/`. */
-    base: string;
-    /** Dev server port. */
-    port: number;
-    /** HMR port. Explicit, NOT `port + 1`: career goes DOWN to 9998 where the others go up. */
-    hmrPort: number;
-    router: RouterPaths;
-    tanstack: TanstackFactories;
-    /**
-     * Sourcemaps: a boolean, however the app decided it.
-     *
-     * The three apps disagree on how — two use a mode allowlist, one reads an env var. That is
-     * an inconsistency in the repo rather than a requirement, and encoding both would make
-     * sentinel the owner of a decision it should not hold.
-     */
-    sourcemap?: boolean;
-    /** Passed through verbatim, never read. See the note above. */
-    alias?: unknown[];
-    /** The app's own plugins (branding, babel). Appended AFTER the base set. */
-    plugins?: PluginOption[];
-    /** Conventions the app may override. */
-    target?: string;
-    allowedHosts?: string[];
-    /**
-     * Anything else this app needs, merged over the preset LAST.
-     *
-     * The preset is a starting point, not a cage. Two things an app must be able to do and
-     * could not before: override a NESTED attribute without losing its siblings, and add an
-     * attribute the preset knows nothing about.
-     *
-     * Objects merge, everything else replaces — see `merge-config` for why arrays replace
-     * rather than concatenate. Whatever lands here is reported by `--inspect --build`, so an
-     * override is a visible, countable decision instead of something found by reading a config.
-     *
-     * Real cases today: console needs `ssr`, `optimizeDeps` and `preview`; career needs
-     * `css.preprocessorOptions` for Sass; all three need their own `nitro.publicAssets`.
-     */
-    overrides?: Record<string, unknown>;
-}
-
-/**
- * The environment, loaded the way all three apps already load it.
- *
- * `production` is normalised to `prd` because that is this repository's deployment tag: it is
- * the `.env.prd` the build reads and the SSM path segment the values come from. Not a
- * preference, a fact about the repo — which is exactly why it belongs in the preset rather
- * than being retyped in every config.
- */
-declare function appEnv(root: string, mode: string): Record<string, string>;
-/** True when this run is a test rather than a real build. */
-declare function isTestMode(mode: string, env?: NodeJS.ProcessEnv): boolean;
-/**
- * WHICH plugins run, as names, in order.
- *
- * Split out from building them so the rule can be tested without instantiating a single
- * plugin. The rule is the part that must not diverge between apps; the instances are just
- * function calls.
- *
- * Under test the router plugin runs alone: `tanstackStart` and `nitro` bring a server the
- * tests neither need nor can run. All three apps already do this, identically.
- */
-declare function pluginPlan(isTest: boolean, extras?: number): string[];
-/**
- * The Vite config for a React app, minus the plugins.
- *
- * Pure and separately testable: everything here is data in, data out, so the composition
- * rules can be asserted without a bundler anywhere near the test.
- */
-declare function reactAppConfig(options: ReactAppOptions): Omit<UserConfig, 'plugins'>;
-/** The config an app spreads into `defineConfig`. */
-declare function reactApp(options: ReactAppOptions): UserConfig;
-
-export { REACT_APP_DEFAULTS, type ReactAppOptions, type RouterPaths, type TanstackFactories, appEnv, isTestMode, pluginPlan, reactApp, reactAppConfig };

package/dist/roles/build/react-app.js

@@ -1,111 +0,0 @@
-import {
-  REACT_APP_DEFAULTS,
-  isPlainObject
-} from "../../chunk-XSCDJZDY.js";
-
-// src/roles/build/react-app.ts
-import tailwindcss from "@tailwindcss/vite";
-import react from "@vitejs/plugin-react";
-import { nitro } from "nitro/vite";
-import { loadEnv } from "vite";
-import svgr from "vite-plugin-svgr";
-
-// src/roles/build/merge-config.ts
-function mergeConfig(base, overrides) {
-  if (!overrides) return base;
-  const out = { ...base };
-  for (const [key, value] of Object.entries(overrides)) {
-    if (value === void 0) {
-      delete out[key];
-      continue;
-    }
-    const current = out[key];
-    out[key] = isPlainObject(current) && isPlainObject(value) ? mergeConfig(current, value) : value;
-  }
-  return out;
-}
-
-// src/roles/build/react-app.ts
-import { defineConfig, loadEnv as loadEnv2 } from "vite";
-function appEnv(root, mode) {
-  return loadEnv(mode === "production" ? "prd" : mode, root, "");
-}
-function isTestMode(mode, env = process.env) {
-  return mode === "test" || env.VITEST === "true";
-}
-function pluginPlan(isTest, extras = 0) {
-  return [
-    "tailwindcss",
-    ...isTest ? ["tanstackRouter"] : ["tanstackStart", "nitro"],
-    "react",
-    "svgr",
-    ...Array.from({ length: extras }, (_, at) => `app:${at}`)
-  ];
-}
-function basePlugins(options, isTest) {
-  const { tanstack, router } = options;
-  return [
-    tailwindcss(),
-    ...isTest ? [tanstack.router({ ...router, target: "react", autoCodeSplitting: true })] : [...tanstack.start({ router }), nitro()],
-    react(),
-    svgr(),
-    ...options.plugins ?? []
-  ];
-}
-function reactAppConfig(options) {
-  const preset = {
-    base: options.base,
-    root: options.root,
-    define: {
-      // Development only. In a build this would leak a browser global into SSR output.
-      ...options.mode === "development" ? { global: "window" } : {}
-    },
-    server: {
-      port: options.port,
-      allowedHosts: options.allowedHosts ?? [...REACT_APP_DEFAULTS.allowedHosts],
-      hmr: { host: "localhost", protocol: "ws", port: options.hmrPort }
-    },
-    build: {
-      target: options.target ?? REACT_APP_DEFAULTS.target,
-      sourcemap: options.sourcemap ?? false
-    },
-    /*
-     * Nitro's two settings are sentinel's, and both were measured across the three apps rather
-     * than assumed.
-     *
-     * `baseURL` is always the app's `base`, without exception: `/console/`, `/` and `/admin/`.
-     * It is derived, so an app cannot get them out of step.
-     *
-     * `publicAssets` is byte-identical in all three — the branding directory, its URL and a
-     * one-year max-age. That is a convention about where branding lives, not data about an
-     * app, and it was sitting in three `overrides` blocks saying the same thing.
-     *
-     * An app that needs MORE from nitro overrides the key: career adds `externals` to keep the
-     * SSR sanitisation chain out of the server bundle. `mergeConfig` merges objects, so adding
-     * one key does not take these two away.
-     */
-    nitro: {
-      baseURL: options.base,
-      publicAssets: [...REACT_APP_DEFAULTS.publicAssets]
-    },
-    ...options.alias ? { resolve: { alias: options.alias } } : {}
-  };
-  return mergeConfig(preset, options.overrides);
-}
-function reactApp(options) {
-  return {
-    ...reactAppConfig(options),
-    plugins: basePlugins(options, isTestMode(options.mode))
-  };
-}
-export {
-  REACT_APP_DEFAULTS,
-  appEnv,
-  defineConfig,
-  isTestMode,
-  loadEnv2 as loadEnv,
-  pluginPlan,
-  reactApp,
-  reactAppConfig
-};
-//# sourceMappingURL=react-app.js.map