@luckystack/devkit 0.13.0 → 0.15.0

package/AGENTS.md

@@ -6,4 +6,4 @@ Generated route contracts must preserve transport-safe types. A source file miss
 
 The one deliberate diagnostics exception — an absent optional sync side — must stay scoped to that side's own field and run after the thrown-extraction check (ADR 0057). Server-backed sync routes share the API side's auth/rate-limit extractors and schema converter so the test-runner can sweep them (ADR 0064). See `docs/type-map-generation.md`.
 
-Hot reload should not block serving live requests while building developer artifacts. Avoid invalidating the shared Program on initial population without cause; do invalidate when source changes. Consumer-owned template overrides intentionally outlive the installed devkit's bundled defaults; do not overwrite them on upgrade.
+Hot reload does pause request handling while it builds the TypeScript Program and regenerates the type map: that work is synchronous, and deferring it only lets the current event finish. The pause is accepted for development; keep its duration logged and keep generation off the boot path so the server listens first. Do not describe it as non-blocking (see `docs/hot-reload.md`). Avoid invalidating the shared Program on initial population without cause; do invalidate when source changes. Consumer-owned template overrides intentionally outlive the installed devkit's bundled defaults; do not overwrite them on upgrade.

package/CHANGELOG.md

@@ -5,6 +5,33 @@ All notable changes to `@luckystack/devkit` are documented in this file.
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.1.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+## [0.15.0] - 2026-10-19
+
+### Changed
+
+- **Editing a file that a route imports restarts the supervised dev server** (`npm run server`), so the new code actually runs; connected clients reconnect (ADR 0079). Before, the route was re-imported but kept running the old dependency code. The supervisor now starts the dev server with an IPC channel for this (Node; Bun on Linux and macOS). Without the supervisor, a warning says a restart is needed. Editing a route file itself still reloads it in place.
+- **Hot reload logs how long type-map regeneration and each route reload took.** The docs now say the dev server answers no requests during that time (ADR 0080).
+
+### Fixed
+
+- **The dev server starts, and type generation runs, when a `.test.ts` or `.spec.ts` file sits next to a route.**
+- **A project checked out under a folder named `_api` or `_sync` boots again.** Route naming is validated relative to the source folder.
+- **The duplicate-route check reports a root sync** (`src/_sync/x_server_v1.ts`) that collides with `src/system/_sync/x_server_v1.ts`, as it already did for APIs.
+- **Deleting a sync `_server` file moves the real `clientInput` type into the paired client** instead of an empty placeholder, and that client rewrite is no longer lost when another sync file is saved within the reload debounce window.
+- **Helpers in a private folder under `_api`/`_sync`** (for example `_api/_lib/helpers.ts`) **and test files next to routes are no longer treated as misnamed routes:** a new empty helper gets no guidance comment written into it, there is no false "ignored by route loading" warning, and editing a helper regenerates types and reloads the routes that use it.
+- **The paired-sync rewrite writes the correct `PagePath`** when the project lives under a folder named `src`, uses a different source folder, or uses a custom sync marker.
+- **A fully commented-out route or page file that arrives by rename, move, `git checkout` or `stash pop` is no longer overwritten with the starter template.** Only empty files, or files that still hold only the injector's guidance line, receive a template.
+- **Route reloads after a dependency change honour a custom source folder and custom route markers.**
+- **Writing the generated Zod schema file no longer triggers a second full type-map generation.**
+- **Saving a watched file while the supervisor waits to restart a crashed server no longer starts two server processes.**
+
+## [0.14.0] - 2026-10-16
+
+### Fixed
+
+- **Injected route and page files no longer start with a blanket `eslint-disable`.** The dev watcher copied the template's own lint-disable header into each new file, which silenced all lint there and, once the file was clean, failed lint as an unused directive. The header is now dropped on injection, for the bundled templates and for project copies in `.luckystack/templates/`; the template files keep it, and a template saved with a byte-order mark loses it too. Files injected earlier keep their header until you delete those two lines.
+- **Injected route and page files pass the scaffold's lint.** Without the header the template bodies failed it: an empty `({ })` parameter, an `async` sync handler without `await`, unused page props, a stray `@ts-ignore` in the page templates and untranslated placeholder text. `main` now takes no parameters (a comment shows what to destructure), the legacy `sync_client` template returns `MaybePromise`, pages take no props until they need them. The empty `{}` input/output shapes stay, because the type generator and the sync-pairing rewrite read them; the scaffold's ESLint config allows them in `_api` and `_sync` files (`create-luckystack-app` 0.14.0). In an existing project, `npx luckystack update` brings the new `.luckystack/templates/` copies and `eslint.official.config.js` (an edited copy gets a `.new` sidecar to merge).
+
 ## [0.12.0] - 2026-10-15
 
 ### Changed

package/dist/chunk-FWE6YJ3G.js

@@ -0,0 +1,17 @@
+// src/devServerRestart.ts
+var DEV_RESTART_MESSAGE_TYPE = "luckystack:dev-restart";
+var isDevRestartMessage = (value) => typeof value === "object" && value !== null && "type" in value && value.type === DEV_RESTART_MESSAGE_TYPE && "reason" in value && typeof value.reason === "string";
+var requestDevServerRestart = (reason) => {
+  if (process.env.LUCKYSTACK_CORE_SUPERVISED !== "true") return false;
+  if (typeof process.send !== "function" || !process.connected) return false;
+  const message = { type: DEV_RESTART_MESSAGE_TYPE, reason };
+  return process.send(message, void 0, void 0, (error) => {
+    if (error) console.log(`[HotReload] Could not ask the supervisor to restart: ${String(error)}`, "red");
+  });
+};
+
+export {
+  isDevRestartMessage,
+  requestDevServerRestart
+};
+//# sourceMappingURL=chunk-FWE6YJ3G.js.map

package/dist/chunk-FWE6YJ3G.js.map

@@ -0,0 +1 @@
+{"version":3,"sources":["../src/devServerRestart.ts"],"sourcesContent":["//? Restart request from the dev server (child) to the supervisor (parent).\n//?\n//? Why a restart and not a reload: hot reload re-imports a route with a `?v=`\n//? query, but the ESM loader caches the route's own imports by their plain URL.\n//? After a helper the route imports changes, the re-imported route still links\n//? to the cached OLD helper, so only a fresh process runs the new code. No\n//? loader hook can carry the query down the import graph under both tsx and\n//? Bun, so the supervisor restarts the child, the same way it does for a\n//? `config.ts` edit.\n//?\n//? This module is bundled into the supervisor entry, so it must import nothing\n//? from `@luckystack/core` (see the env invariant in `supervisor.ts`).\n\nexport const DEV_RESTART_MESSAGE_TYPE = 'luckystack:dev-restart';\n\nexport interface DevRestartMessage {\n  type: typeof DEV_RESTART_MESSAGE_TYPE;\n  reason: string;\n}\n\nexport const isDevRestartMessage = (value: unknown): value is DevRestartMessage =>\n  typeof value === 'object'\n  && value !== null\n  && 'type' in value\n  && value.type === DEV_RESTART_MESSAGE_TYPE\n  && 'reason' in value\n  && typeof value.reason === 'string';\n\n/**\n * Ask the supervisor to restart this dev server. Returns false when nothing\n * can restart it (started without the supervisor, or no IPC channel), so the\n * caller can tell the developer to restart by hand.\n */\nexport const requestDevServerRestart = (reason: string): boolean => {\n  if (process.env.LUCKYSTACK_CORE_SUPERVISED !== 'true') return false;\n  if (typeof process.send !== 'function' || !process.connected) return false;\n\n  const message: DevRestartMessage = { type: DEV_RESTART_MESSAGE_TYPE, reason };\n  //? The callback keeps a failed send from surfacing as an unhandled 'error'.\n  return process.send(message, undefined, undefined, (error: Error | null) => {\n    if (error) console.log(`[HotReload] Could not ask the supervisor to restart: ${String(error)}`, 'red');\n  });\n};\n"],"mappings":";AAaO,IAAM,2BAA2B;AAOjC,IAAM,sBAAsB,CAAC,UAClC,OAAO,UAAU,YACd,UAAU,QACV,UAAU,SACV,MAAM,SAAS,4BACf,YAAY,SACZ,OAAO,MAAM,WAAW;AAOtB,IAAM,0BAA0B,CAAC,WAA4B;AAClE,MAAI,QAAQ,IAAI,+BAA+B,OAAQ,QAAO;AAC9D,MAAI,OAAO,QAAQ,SAAS,cAAc,CAAC,QAAQ,UAAW,QAAO;AAErE,QAAM,UAA6B,EAAE,MAAM,0BAA0B,OAAO;AAE5E,SAAO,QAAQ,KAAK,SAAS,QAAW,QAAW,CAAC,UAAwB;AAC1E,QAAI,MAAO,SAAQ,IAAI,wDAAwD,OAAO,KAAK,CAAC,IAAI,KAAK;AAAA,EACvG,CAAC;AACH;","names":[]}

package/dist/devServerRestart.d.ts

@@ -0,0 +1,12 @@
+export declare const DEV_RESTART_MESSAGE_TYPE = "luckystack:dev-restart";
+export interface DevRestartMessage {
+    type: typeof DEV_RESTART_MESSAGE_TYPE;
+    reason: string;
+}
+export declare const isDevRestartMessage: (value: unknown) => value is DevRestartMessage;
+/**
+ * Ask the supervisor to restart this dev server. Returns false when nothing
+ * can restart it (started without the supervisor, or no IPC channel), so the
+ * caller can tell the developer to restart by hand.
+ */
+export declare const requestDevServerRestart: (reason: string) => boolean;