@plumeria/init 19.9.1 → 19.10.0

package/README.md

@@ -13,11 +13,11 @@ writes. Nothing is installed or patched before you say so.
 ```
 ✔ detected  next  pnpm  typescript
 
-  ➡︎ install  @plumeria/core @plumeria/next-plugin @plumeria/eslint-plugin oxlint
+  ➡︎ install  @plumeria/core @plumeria/next-plugin @plumeria/eslint-plugin rimraf
   + write    plumeria.d.ts     classStyle
   ~ patch    next.config.ts    withPlumeria(...)
   ~ patch    eslint.config.ts  plumeria.configs.recommended
-  ~ patch    package.json      build: plumerialint -- next build
+  ~ patch    package.json      predev: rimraf .next, prebuild: rimraf .next
 ```
 
 ## What it detects
@@ -42,20 +42,17 @@ so the counterpart cannot compile, and ESLint gets
 is reported while you type. Answering "none" sets neither. `--sizes` extends
 the answer to the size axis, where `width` and `inlineSize` are the pair.
 
-**Expand a border shorthand into the three declarations it sets?** — turns on
-`@plumeria/expand-border-shorthands`. A `border` shorthand crosses the axis
-shorthands without either containing the other, so leaving it whole makes the
-outcome depend on order.
-
 **Which JSX prop carries styles?** — `classStyle` by default. A renamed prop is
 written into the plugin as `styleProp` and declared in `plumeria.d.ts`, so the
 two cannot disagree. It has to be an identifier TypeScript can declare, and a
 name React leaves free; `className`, `style`, `key`, `ref` and `children` are
 refused, from the prompt and from `--style-prop` alike.
 
-**Set up `@plumeria/eslint-plugin` and the `plumerialint` build guard?** — adds
-the plugin and `oxlint`, writes or extends the flat config, and prefixes the
-`build` script with `plumerialint --`, so a style error stops the build.
+**Set up `@plumeria/eslint-plugin` for ESLint?** — adds the plugin and writes or
+extends the flat config, so a style error is reported while you type. The build
+lints either way: the bundler plugin runs the same rules alongside the build and
+stops it on a style error. On esbuild and Bun, which the plugin cannot lint,
+the `build` script is prefixed with `plumerialint --` instead.
 
 ## What it writes
 
@@ -66,8 +63,8 @@ the plugin and `oxlint`, writes or extends the flat config, and prefixes the
   wrapped in `withPlumeria`.
 - `eslint.config.ts` — `plumeria.configs.recommended` plus the rules that were
   asked for. An existing flat config is extended rather than replaced.
-- `package.json` — `plumerialint --` in front of the `build` script, and on
-  Next.js `rimraf .next` before `dev` and before `build`, so a version change is
+- `package.json` — on esbuild and Bun, `plumerialint --` in front of the `build`
+  script, and on Next.js `rimraf .next` before `dev` and before `build`, so a version change is
   not read as a compile error. A `pre` script is added only for a script the
   project actually has, and one it already wrote is left alone. Clearing the
   cache is a Next.js concern rather than a lint one, so `--no-eslint` keeps it.
@@ -94,7 +91,7 @@ npx @plumeria/init --yes                # take every default, ask nothing
 npx @plumeria/init --physical --sizes   # answer the spelling question up front
 npx @plumeria/init --style-prop sx
 npx @plumeria/init --bundler rollup
-npx @plumeria/init --no-eslint          # leave ESLint and the plumerialint guard out
+npx @plumeria/init --no-eslint          # leave ESLint out
 npx @plumeria/init --no-install         # write the configs, print the command
 npx @plumeria/init --cwd packages/app
 ```

package/dist/cli.js

@@ -29,7 +29,7 @@ Options
       --logical        this project writes logical properties
       --physical       this project writes physical properties
       --sizes          extend the spelling policy to the size axis
-      --no-eslint      leave ESLint and the plumerialint guard out
+      --no-eslint      leave ESLint out
       --no-install     write the configs, print the install command
       --cwd <dir>      run against another directory
   -h, --help           show this message

package/dist/prompt.js

@@ -67,9 +67,6 @@ const ask = async (asker, preset) => {
     if (answers.spelling !== 'none' && preset.sizes === undefined) {
         answers.sizes = await confirm(asker, 'Reject the size axis too (width/height ↔ inlineSize/blockSize)?', false);
     }
-    if (preset.expandBorderShorthands === undefined) {
-        answers.expandBorderShorthands = await confirm(asker, 'Expand a border shorthand into the three declarations it sets?', true);
-    }
     if (preset.styleProp === undefined) {
         const answer = (await asker.question(`Which JSX prop carries styles? ${style_1.style.faint(`[${setup_1.DEFAULT_STYLE_PROP}]`)} `)).trim();
         if (answer !== '') {
@@ -77,7 +74,7 @@ const ask = async (asker, preset) => {
         }
     }
     if (preset.eslint === undefined) {
-        answers.eslint = await confirm(asker, 'Set up @plumeria/eslint-plugin and the plumerialint build guard?', true);
+        answers.eslint = await confirm(asker, 'Set up @plumeria/eslint-plugin for ESLint?', true);
     }
     return answers;
 };

package/dist/setup.d.ts

@@ -4,7 +4,6 @@ export interface Answers {
     spelling: Spelling;
     sizes: boolean;
     styleProp: string;
-    expandBorderShorthands: boolean;
     eslint: boolean;
     install: boolean;
 }

package/dist/setup.js

@@ -23,7 +23,6 @@ exports.DEFAULT_ANSWERS = {
     spelling: 'none',
     sizes: false,
     styleProp: exports.DEFAULT_STYLE_PROP,
-    expandBorderShorthands: true,
     eslint: true,
     install: true,
 };
@@ -146,12 +145,7 @@ export default defineConfig({
 };
 exports.bundlerSource = bundlerSource;
 const eslintSource = (answers, typescript) => {
-    const rules = [
-        answers.expandBorderShorthands
-            ? `'@plumeria/expand-border-shorthands': 'warn'`
-            : undefined,
-        (0, exports.spellingRule)(answers),
-    ].filter((rule) => rule !== undefined);
+    const rules = [(0, exports.spellingRule)(answers)].filter((rule) => rule !== undefined);
     const block = [
         `  {`,
         `    files: ['**/*.{${typescript ? 'ts,tsx' : 'js,jsx'}}'],`,
@@ -184,12 +178,7 @@ ${block.join('\n')}
 };
 exports.eslintSource = eslintSource;
 const eslintEntries = (answers, typescript) => {
-    const rules = [
-        answers.expandBorderShorthands
-            ? `'@plumeria/expand-border-shorthands': 'warn'`
-            : undefined,
-        (0, exports.spellingRule)(answers),
-    ].filter((rule) => rule !== undefined);
+    const rules = [(0, exports.spellingRule)(answers)].filter((rule) => rule !== undefined);
     const entries = ['plumeria.configs.recommended'];
     if (rules.length > 0) {
         entries.push([
@@ -204,6 +193,7 @@ const eslintEntries = (answers, typescript) => {
     return entries;
 };
 exports.eslintEntries = eslintEntries;
+const guardsByScript = (detected) => detected.bundler === 'esbuild' || detected.bundler === 'bun';
 const packages = (detected, answers) => {
     const present = (0, detect_1.installed)(detected.manifest);
     const wanted = [
@@ -213,7 +203,7 @@ const packages = (detected, answers) => {
             : '@plumeria/unplugin',
     ];
     if (answers.eslint) {
-        wanted.push('@plumeria/eslint-plugin', 'oxlint');
+        wanted.push('@plumeria/eslint-plugin');
         if (!detected.eslintConfig) {
             wanted.push('eslint', '@eslint/js');
             if (detected.typescript)
@@ -267,7 +257,8 @@ const plan = (detected, answers) => {
     actions.push(bundlerAction(detected, answers));
     if (answers.eslint)
         actions.push(eslintAction(detected, answers));
-    if (answers.eslint || detected.bundler === 'next')
+    if ((answers.eslint && guardsByScript(detected)) ||
+        detected.bundler === 'next')
         actions.push(scriptAction(detected, answers));
     return actions;
 };
@@ -464,7 +455,8 @@ const scriptAction = (detected, answers) => {
     const scripts = { ...manifest.scripts };
     const notes = [];
     const existing = scripts.build;
-    if (answers.eslint && existing && !existing.includes('plumerialint')) {
+    const guard = answers.eslint && guardsByScript(detected);
+    if (guard && existing && !existing.includes('plumerialint')) {
         scripts.build = (0, exports.buildScript)(existing);
         notes.push(`build: ${scripts.build}`);
     }
@@ -479,7 +471,7 @@ const scriptAction = (detected, answers) => {
         return {
             kind: 'skip',
             label: 'skip',
-            note: answers.eslint && !existing
+            note: guard && !existing
                 ? 'no build script to guard'
                 : 'the scripts already carry it',
             file: 'package.json',

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@plumeria/init",
-  "version": "19.9.1",
+  "version": "19.10.0",
   "description": "One command that sets up Plumeria: detects the bundler, installs the packages, writes the configs",
   "author": "Refirst 11",
   "license": "MIT",