@fluid-app/fluid-cli-theme-dev 0.1.19 → 0.1.21
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
- package/.turbo/turbo-build.log +5 -5
- package/.turbo/turbo-typecheck.log +1 -1
- package/dist/index.mjs +6 -2
- package/dist/index.mjs.map +1 -1
- package/package.json +4 -4
- package/skills/themes-review/SKILL.md +707 -0
- package/skills/themes-review/references/blocks-vs-sections.md +131 -0
- package/skills/themes-review/references/css-js-hygiene.md +153 -0
- package/skills/themes-review/references/dead-code.md +161 -0
- package/skills/themes-review/references/dynamism.md +88 -0
- package/skills/themes-review/references/editor-attributes.md +160 -0
- package/skills/themes-review/references/examples.md +111 -0
- package/skills/themes-review/references/fairshare-attributes.md +185 -0
- package/skills/themes-review/references/global-settings.md +221 -0
- package/skills/themes-review/references/liquid-correctness.md +176 -0
- package/skills/themes-review/references/navigation.md +88 -0
- package/skills/themes-review/references/performance.md +85 -0
- package/skills/themes-review/references/security-accessibility.md +70 -0
- package/skills/themes-review/references/setting-types.md +118 -0
- package/src/commands/lint.ts +18 -2
package/.turbo/turbo-build.log
CHANGED
|
@@ -1,5 +1,5 @@
|
|
|
1
1
|
|
|
2
|
-
> @fluid-app/fluid-cli-theme-dev@0.1.
|
|
2
|
+
> @fluid-app/fluid-cli-theme-dev@0.1.21 build /home/runner/_work/fluid-mono/fluid-mono/packages/cli/theme-dev
|
|
3
3
|
> tsdown
|
|
4
4
|
|
|
5
5
|
[34mℹ[39m tsdown [2mv0.21.0[22m powered by rolldown [2mv1.0.0-rc.7[22m
|
|
@@ -8,9 +8,9 @@
|
|
|
8
8
|
[34mℹ[39m target: [34mnode24[39m
|
|
9
9
|
[34mℹ[39m tsconfig: [34mtsconfig.json[39m
|
|
10
10
|
[34mℹ[39m Build start
|
|
11
|
-
[34mℹ[39m [2mdist/[22m[1mindex.mjs[22m [2m 76.
|
|
12
|
-
[34mℹ[39m [2mdist/[22mindex.mjs.map [
|
|
11
|
+
[34mℹ[39m [2mdist/[22m[1mindex.mjs[22m [2m 76.86 kB[22m [2m│ gzip: 21.05 kB[22m
|
|
12
|
+
[34mℹ[39m [2mdist/[22mindex.mjs.map [2m196.02 kB[22m [2m│ gzip: 46.38 kB[22m
|
|
13
13
|
[34mℹ[39m [2mdist/[22mindex.d.mts.map [2m 0.11 kB[22m [2m│ gzip: 0.12 kB[22m
|
|
14
14
|
[34mℹ[39m [2mdist/[22m[32m[1mindex.d.mts[22m[39m [2m 0.19 kB[22m [2m│ gzip: 0.16 kB[22m
|
|
15
|
-
[34mℹ[39m 4 files, total:
|
|
16
|
-
[32m✔[39m Build complete in [
|
|
15
|
+
[34mℹ[39m 4 files, total: 273.18 kB
|
|
16
|
+
[32m✔[39m Build complete in [32m1315ms[39m
|
package/dist/index.mjs
CHANGED
|
@@ -526,10 +526,12 @@ const VALID_SETTING_TYPES = Object.values({
|
|
|
526
526
|
/**
|
|
527
527
|
* Message shown when a setting declares a `type` that is not one of the
|
|
528
528
|
* canonical `VALID_SETTING_TYPES`. Centralized here so the CLI and the editor
|
|
529
|
-
* render identical text
|
|
529
|
+
* render identical text. Kept to a single line — the list of valid types is a
|
|
530
|
+
* static set exposed once via the `VALID_SETTING_TYPES` export, so repeating it
|
|
531
|
+
* in every diagnostic only bloats structured output.
|
|
530
532
|
*/
|
|
531
533
|
function invalidSettingTypeMessage(type) {
|
|
532
|
-
return `Invalid settings type: '${type}'
|
|
534
|
+
return `Invalid settings type: '${type}'`;
|
|
533
535
|
}
|
|
534
536
|
function validateSettings(settings) {
|
|
535
537
|
const diagnostics = [];
|
|
@@ -2108,11 +2110,13 @@ function createLintCommand() {
|
|
|
2108
2110
|
let warnings = 0;
|
|
2109
2111
|
for (const { diagnostics } of results) for (const d of diagnostics) if (d.severity === "error") errors++;
|
|
2110
2112
|
else warnings++;
|
|
2113
|
+
const hasInvalidSettingType = results.some(({ diagnostics }) => diagnostics.some((d) => d.target?.kind === "setting" && d.target.field === "type" && d.target.settingType !== void 0));
|
|
2111
2114
|
if (opts.json) console.log(JSON.stringify({
|
|
2112
2115
|
ok: errors === 0,
|
|
2113
2116
|
errors,
|
|
2114
2117
|
warnings,
|
|
2115
2118
|
filesChecked: liquidFiles.length,
|
|
2119
|
+
...hasInvalidSettingType ? { validSettingTypes: VALID_SETTING_TYPES } : {},
|
|
2116
2120
|
files: results
|
|
2117
2121
|
}));
|
|
2118
2122
|
else printText(results, errors, warnings, liquidFiles.length);
|