@fluid-app/fluid-cli-theme-dev 0.1.18 → 0.1.20

package/.turbo/turbo-build.log

@@ -1,5 +1,5 @@
 
-> @fluid-app/fluid-cli-theme-dev@0.1.18 build /home/runner/_work/fluid-mono/fluid-mono/packages/cli/theme-dev
+> @fluid-app/fluid-cli-theme-dev@0.1.20 build /home/runner/_work/fluid-mono/fluid-mono/packages/cli/theme-dev
 > tsdown
 
 ℹ tsdown v0.21.0 powered by rolldown v1.0.0-rc.7
@@ -8,9 +8,9 @@
 ℹ target: node24
 ℹ tsconfig: tsconfig.json
 ℹ Build start
-ℹ dist/index.mjs         75.84 kB │ gzip: 20.63 kB
-ℹ dist/index.mjs.map    193.35 kB │ gzip: 45.34 kB
+ℹ dist/index.mjs         76.86 kB │ gzip: 21.05 kB
+ℹ dist/index.mjs.map    196.02 kB │ gzip: 46.38 kB
 ℹ dist/index.d.mts.map    0.11 kB │ gzip:  0.12 kB
 ℹ dist/index.d.mts        0.19 kB │ gzip:  0.16 kB
-ℹ 4 files, total: 269.50 kB
-✔ Build complete in 2984ms
+ℹ 4 files, total: 273.18 kB
+✔ Build complete in 1274ms

package/.turbo/turbo-typecheck.log

@@ -1,4 +1,4 @@
 
-> @fluid-app/fluid-cli-theme-dev@0.1.18 typecheck /home/runner/_work/fluid-mono/fluid-mono/packages/cli/theme-dev
+> @fluid-app/fluid-cli-theme-dev@0.1.20 typecheck /home/runner/_work/fluid-mono/fluid-mono/packages/cli/theme-dev
 > tsc --noEmit
 

package/dist/index.mjs

@@ -526,10 +526,12 @@ const VALID_SETTING_TYPES = Object.values({
 /**
 * Message shown when a setting declares a `type` that is not one of the
 * canonical `VALID_SETTING_TYPES`. Centralized here so the CLI and the editor
-* render identical text (including the list of available types).
+* render identical text. Kept to a single line — the list of valid types is a
+* static set exposed once via the `VALID_SETTING_TYPES` export, so repeating it
+* in every diagnostic only bloats structured output.
 */
 function invalidSettingTypeMessage(type) {
-	return `Invalid settings type: '${type}'\nAvailable types:\n${VALID_SETTING_TYPES.join(", \n")}`;
+	return `Invalid settings type: '${type}'`;
 }
 function validateSettings(settings) {
 	const diagnostics = [];
@@ -740,6 +742,16 @@ function validateSchemaText(text, options) {
 const LIQUID_COMMENT_REGEX = /\{%-?\s*comment\s*-?%\}[\s\S]*?\{%-?\s*endcomment\s*-?%\}/g;
 const SCHEMA_BLOCK_REGEX = /\{%-?\s*schema\s*-?%\}[\s\S]*?\{%-?\s*endschema\s*-?%\}/;
 const SECTION_TAG_PATTERN = "\\{%-?\\s*section\\s+['\"]([^'\"]+)['\"](?:\\s*,\\s*id:\\s*['\"]([^'\"]+)['\"])?\\s*-?%\\}";
+const RESERVED_SECTION_TYPES = new Set([
+	"navbar",
+	"library_navbar",
+	"footer"
+]);
+const EXTENSION_URI_PATTERN = /^fluid:\/\/extensions\/[^/]+\/[^/]+\/[^/]+$/;
+/** Whether a `{% section %}` type resolves to something other than an on-disk `sections/<name>` definition (and so cannot be flagged as missing). */
+function isNonLocalSectionType(type) {
+	return RESERVED_SECTION_TYPES.has(type) || EXTENSION_URI_PATTERN.test(type);
+}
 function templateBody(liquid) {
 	return liquid.replace(LIQUID_COMMENT_REGEX, "").replace(SCHEMA_BLOCK_REGEX, "");
 }
@@ -770,7 +782,9 @@ function extractSectionReferences(liquid) {
 /**
 * Find `{% section %}` references that point to a section that does not exist
 * in `existingSectionNames` — the static equivalent of "an in-use section was
-* deleted". Emits one `error` diagnostic per missing section type per template.
+* deleted". Reserved layout-region types (navbar/footer/library_navbar) and
+* `fluid://` extension URIs are never flagged (see `isNonLocalSectionType`).
+* Emits one `error` diagnostic per missing section type per template.
 */
 function findMissingSectionReferences(templates, existingSectionNames) {
 	const missing = [];
@@ -778,6 +792,7 @@ function findMissingSectionReferences(templates, existingSectionNames) {
 		const reported = /* @__PURE__ */ new Set();
 		for (const reference of extractSectionReferences(template.content)) {
 			if (existingSectionNames.has(reference.type)) continue;
+			if (isNonLocalSectionType(reference.type)) continue;
 			if (reported.has(reference.type)) continue;
 			reported.add(reference.type);
 			missing.push({
@@ -799,6 +814,15 @@ function findMissingSectionReferences(templates, existingSectionNames) {
 }
 //#endregion
 //#region src/theme/file.ts
+const NON_TEMPLATE_DIRS = new Set([
+	"sections",
+	"blocks",
+	"components",
+	"layouts",
+	"config",
+	"assets",
+	"locales"
+]);
 var ThemeFile = class {
 	absolutePath;
 	relativePath;
@@ -843,7 +867,7 @@ var ThemeFile = class {
 	}
 	get isTemplate() {
 		const parts = this.relativePath.split(/[/\\]/);
-		return parts[0] === "templates" && parts.length >= 3 && parts[1] !== "sections" && parts[1] !== "blocks" && parts[1] !== "components";
+		return parts.length >= 2 && !NON_TEMPLATE_DIRS.has(parts[0]);
 	}
 	validateSchema() {
 		if (!this.isLiquid) return [];
@@ -2034,7 +2058,7 @@ function createPullCommand() {
 //#region src/commands/lint.ts
 function sectionNameOf(relativePath) {
 	const parts = relativePath.split(/[/\\]/);
-	if (parts[0] === "templates" && parts[1] === "sections" && parts.length >= 3) return parts[2].replace(/\.liquid$/, "");
+	if (parts[0] === "sections" && parts.length >= 2) return parts[1].replace(/\.liquid$/, "");
 	return null;
 }
 function createLintCommand() {
@@ -2086,11 +2110,13 @@ function createLintCommand() {
 		let warnings = 0;
 		for (const { diagnostics } of results) for (const d of diagnostics) if (d.severity === "error") errors++;
 		else warnings++;
+		const hasInvalidSettingType = results.some(({ diagnostics }) => diagnostics.some((d) => d.target?.kind === "setting" && d.target.field === "type" && d.target.settingType !== void 0));
 		if (opts.json) console.log(JSON.stringify({
 			ok: errors === 0,
 			errors,
 			warnings,
 			filesChecked: liquidFiles.length,
+			...hasInvalidSettingType ? { validSettingTypes: VALID_SETTING_TYPES } : {},
 			files: results
 		}));
 		else printText(results, errors, warnings, liquidFiles.length);