@luckystack/devkit 0.13.0 → 0.14.0

package/CHANGELOG.md

@@ -5,6 +5,13 @@ 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.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/index.js

@@ -4458,10 +4458,16 @@ var checkPagePlacement = (filePath, srcRelativePath) => {
   }
   return null;
 };
+var TEMPLATE_LINT_HEADER = /^(?:\/\* eslint-disable(?: unicorn\/no-abusive-eslint-disable)? \*\/[^\S\r\n]*\r?\n)+(?:[^\S\r\n]*\r?\n)?/;
+var BYTE_ORDER_MARK = String.fromCodePoint(65279);
+var stripTemplateLintHeader = (content) => {
+  const unmarked = content.startsWith(BYTE_ORDER_MARK) ? content.slice(1) : content;
+  return TEMPLATE_LINT_HEADER.test(unmarked) ? unmarked.replace(TEMPLATE_LINT_HEADER, "") : content;
+};
 var applyTemplatePlaceholders = (content, filePath, fileKind) => {
   const relPath = calculateRelativePath(filePath);
   const pragmaPattern = /\/\/\s*@ts-(?:ignore|expect-error).*\r?\n(.*)\{\{REL_PATH\}\}/g;
-  let result = content.replaceAll(pragmaPattern, (_, prefix) => `${prefix}${relPath}`);
+  let result = stripTemplateLintHeader(content).replaceAll(pragmaPattern, (_, prefix) => `${prefix}${relPath}`);
   result = result.replaceAll("{{REL_PATH}}", () => relPath);
   if (fileKind === "sync_client" || fileKind === "sync_server") {
     const pagePath = extractSyncPagePath2(filePath);
@@ -4685,11 +4691,12 @@ var injectServerTemplateWithClientInput = async (serverFilePath, clientInputType
   try {
     await ensureConsumerTemplateConfigLoaded();
     const relPath = calculateRelativePath(serverFilePath);
-    let content = resolveTemplateContent("sync_server");
-    if (content === null) {
+    const rawContent = resolveTemplateContent("sync_server");
+    if (rawContent === null) {
       console.error(`[TemplateInjector] No sync_server template content available for: ${serverFilePath}`);
       return false;
     }
+    let content = stripTemplateLintHeader(rawContent);
     const pragmaPattern = /\/\/\s*@ts-(?:ignore|expect-error).*\r?\n(.*)\{\{REL_PATH\}\}/g;
     content = content.replaceAll(pragmaPattern, (_, prefix) => {
       return `${prefix}${relPath}`;