@elia97/officina 0.5.0 → 0.6.1

package/README.md

@@ -79,6 +79,8 @@ La radice del pacchetto esporta `defineConfig` e i tipi; i pezzi del motore stan
 
 `features` dice quali controlli il progetto pretende: `'required'` fa fallire il gate quando ciò che deve guardare non c'è, `false` lo spegne dicendolo. Una voce non dichiarata vale `'required'`, e `doctor` la segnala: un gate che esce 0 su un container GTM che non esiste non afferma niente. `routes.representatives` dà a ogni pattern dinamico di `src/pages` un percorso vero, e da lì smoke e Lighthouse guardano anche quelle pagine.
 
+`placeholders` dice a `check placeholders` cosa guardare: `sources` sono i file da scandire, `contactEnvKeys` le variabili che `--env` pretende. Una lista dichiarata **sostituisce** quella del pacchetto, dizionari compresi: senza dichiarazione i sorgenti sono `src/lib/site.ts`, `src/lib/company.ts` e i dizionari di `src/i18n/strings`, quando la cartella esiste. Serve ai progetti che non nascono dallo scaffold: se partita IVA, recapiti e destinatario dei form stanno nel contenuto invece che in un modulo, i sorgenti del template non sono i suoi, e crearli per far passare il gate terrebbe gli stessi dati in due sedi. Un sorgente elencato che non esiste è un ritrovamento con il suo percorso, non un errore che ferma il comando.
+
 Il file lo carica Node, che fuori da `node_modules` toglie i tipi da sé: niente `enum`, niente alias `@/`. Due file `officina.config.*` insieme sono un errore, come un file senza default export; il resto lo valida `loadConfig` a runtime, nominando il percorso della voce sbagliata — `bundle.cssMaxGzip: atteso un numero, ricevuto una stringa` — e una voce sconosciuta, perché `sitUrl` al posto di `siteUrl` non deve passare in silenzio.
 
 `sharp` è una dipendenza facoltativa del progetto, non del pacchetto: serve solo a `gen icons`, che senza lo dice ed esce 1. `@biomejs/biome` invece è una peer dependency con l'intervallo che il preset regge, e `doctor` confronta la versione del progetto con quello.

package/dist/checks/placeholders.d.ts

@@ -1,2 +1,2 @@
 #!/usr/bin/env node
-export declare function main(args?: string[]): number;
+export declare function main(args?: string[]): Promise<number>;

package/dist/checks/placeholders.js

@@ -2,30 +2,36 @@
 import { existsSync, readFileSync } from 'node:fs';
 import process from 'node:process';
 import { parseArgs } from 'node:util';
-import { envFindings, placeholderSources, sourceFindings } from '../lib/check-placeholders.js';
+import { contactEnvKeys, envFindings, placeholderSources, sourceFindings } from '../lib/check-placeholders.js';
 import { cliOptions, exitCode, printFindings } from '../lib/cli.js';
+import { loadConfig } from '../lib/config.js';
+const MISSING_SOURCE = 'sorgente assente: `placeholders.sources` in officina.config.ts dice quali file guardare';
 function sourceScan(paths) {
-    return paths.flatMap((path) => sourceFindings(readFileSync(path, 'utf8')).map((hit) => ({ path, ...hit, severity: 'error' })));
+    return paths.flatMap((path) => {
+        if (!existsSync(path))
+            return [{ path, message: MISSING_SOURCE, severity: 'error' }];
+        return sourceFindings(readFileSync(path, 'utf8')).map((hit) => ({ path, ...hit, severity: 'error' }));
+    });
 }
-function envScan(path) {
+function envScan(path, keys) {
     if (!existsSync(path))
         return [{ path, message: "file dell'ambiente assente: lo scrive vercel pull", severity: 'error' }];
-    return envFindings(readFileSync(path, 'utf8')).map((hit) => ({ path, ...hit, severity: 'error' }));
+    return envFindings(readFileSync(path, 'utf8'), keys).map((hit) => ({ path, ...hit, severity: 'error' }));
 }
-function scan(env) {
+function scan(env, config) {
     if (env !== undefined) {
         return {
             scope: `ambiente in ${env}`,
-            findings: envScan(env),
+            findings: envScan(env, contactEnvKeys(config)),
             fix: 'docs/guides/deploy-ops.md § Checklist per il go-live',
         };
     }
-    const paths = placeholderSources();
+    const paths = placeholderSources(config);
     return { scope: `${paths.length} sorgenti`, findings: sourceScan(paths), fix: 'README.md § Cosa tocca il rebranding' };
 }
-export function main(args = process.argv.slice(2)) {
+export async function main(args = process.argv.slice(2)) {
     const { env } = parseArgs({ args, options: { env: { type: 'string' } } }).values;
-    const { scope, findings, fix } = scan(env);
+    const { scope, findings, fix } = scan(env, await loadConfig(process.cwd()));
     console.log(`\ncheck:placeholders — ${scope}\n`);
     printFindings(findings, cliOptions([]).format);
     if (findings.length === 0) {
@@ -37,4 +43,4 @@ export function main(args = process.argv.slice(2)) {
     return exitCode(findings, false);
 }
 if (import.meta.main)
-    process.exit(main());
+    process.exit(await main());

package/dist/lib/action-manifests.d.ts

@@ -0,0 +1,9 @@
+import { type Hit } from './cli.ts';
+export interface Manifest {
+    path: string;
+    source: string;
+}
+export interface ManifestHit extends Hit {
+    path: string;
+}
+export declare function manifestFindings(manifests: readonly Manifest[]): ManifestHit[];

package/dist/lib/action-manifests.js

@@ -0,0 +1,25 @@
+import { readLines } from './cli.js';
+import { isLive } from './workflows.js';
+const EXPRESSION = /\$\{\{[^}]*\}\}/g;
+const DESCRIPTION = /^\s*description:/;
+const FORBIDDEN = ['secrets', 'needs', 'matrix'];
+const contextIn = (expression) => FORBIDDEN.find((name) => new RegExp(`(^|[^\\w.])${name}\\.`).test(expression));
+function lineFindings(text) {
+    const expressions = [...text.matchAll(EXPRESSION)];
+    if (expressions.length === 0)
+        return [];
+    if (DESCRIPTION.test(text)) {
+        return ['`description` non ammette espressioni: il runner la valuta e il manifesto non si carica'];
+    }
+    return expressions.flatMap(([expression]) => {
+        const context = contextIn(expression);
+        return context === undefined ? [] : [`\`${context}\` non è un contesto di una composite action: passalo come input`];
+    });
+}
+// Il runner valuta le espressioni del manifesto prima di eseguire qualunque passo: un contesto che una
+// composite action non conosce non è un valore vuoto, è un'action che non si carica.
+export function manifestFindings(manifests) {
+    return manifests.flatMap(({ path, source }) => readLines(source)
+        .filter(({ text }) => isLive(text))
+        .flatMap(({ n, text }) => lineFindings(text).map((message) => ({ path, line: n, message }))));
+}

package/dist/lib/check-placeholders.d.ts

@@ -1,6 +1,8 @@
 import { type Finding, type Hit } from './cli.ts';
+import type { OfficinaConfig } from './config.ts';
 type EnvHit = Omit<Finding, 'path' | 'severity'>;
-export declare function placeholderSources(): string[];
+export declare function placeholderSources(config?: OfficinaConfig): string[];
+export declare const contactEnvKeys: (config?: OfficinaConfig) => readonly string[];
 export declare function sourceFindings(source: string): Hit[];
-export declare function envFindings(content: string): EnvHit[];
+export declare function envFindings(content: string, keys: readonly string[]): EnvHit[];
 export {};

package/dist/lib/check-placeholders.js

@@ -1,8 +1,9 @@
-import { readdirSync } from 'node:fs';
+import { existsSync, readdirSync } from 'node:fs';
 import { join } from 'node:path';
 import { parseEnv } from 'node:util';
 import { readLines } from './cli.js';
 const DICTIONARIES = 'src/i18n/strings';
+const DEFAULT_SOURCES = ['src/lib/site.ts', 'src/lib/company.ts'];
 const CONTACT_ENV_KEYS = ['CONTACT_FROM_EMAIL', 'CONTACT_FROM_NAME', 'CONTACT_TO_EMAIL'];
 const TOKEN = /<[A-Z][A-Z_]{2,}>/;
 const DOMAIN = /\bexample\.com\b/;
@@ -10,13 +11,19 @@ const STRING_LITERAL = /'(?:[^'\\]|\\.)*'|"(?:[^"\\]|\\.)*"/g;
 const COMMENT_ONLY = /^\s*(?:\/\/|\/\*|\*)/;
 const ZERO_POSTAL_CODE = /\bpostalCode:\s*(['"])00000\1/;
 const EMPTY_HREF = /\bhref:\s*(['"])#\1/;
-export function placeholderSources() {
-    const dictionaries = readdirSync(DICTIONARIES)
+const dictionaries = () => existsSync(DICTIONARIES)
+    ? readdirSync(DICTIONARIES)
         .filter((name) => name.endsWith('.ts'))
         .sort()
-        .map((name) => join(DICTIONARIES, name));
-    return ['src/lib/site.ts', 'src/lib/company.ts', ...dictionaries];
+        .map((name) => join(DICTIONARIES, name))
+    : [];
+export function placeholderSources(config = {}) {
+    const declared = config.placeholders?.sources;
+    if (declared !== undefined)
+        return [...declared];
+    return [...DEFAULT_SOURCES, ...dictionaries()];
 }
+export const contactEnvKeys = (config = {}) => config.placeholders?.contactEnvKeys ?? CONTACT_ENV_KEYS;
 function literalFindings(value) {
     const messages = [];
     const token = value.match(TOKEN);
@@ -40,10 +47,10 @@ export function sourceFindings(source) {
         return messages.map((message) => ({ line: n, message }));
     });
 }
-export function envFindings(content) {
+export function envFindings(content, keys) {
     const env = parseEnv(content);
     const lines = readLines(content);
-    return CONTACT_ENV_KEYS.flatMap((key) => {
+    return keys.flatMap((key) => {
         const value = env[key]?.trim() ?? '';
         const declaration = lines.find(({ text }) => new RegExp(`^\\s*${key}\\s*=`).test(text));
         const at = declaration ? { line: declaration.n } : {};

package/dist/lib/config.d.ts

@@ -26,6 +26,10 @@ export interface OfficinaConfig {
         analytics?: FeatureSetting;
         roadmap?: FeatureSetting;
     };
+    placeholders?: {
+        sources?: readonly string[];
+        contactEnvKeys?: readonly string[];
+    };
     /** Per ogni pattern dinamico di `src/pages`, un percorso vero che smoke e Lighthouse visitano. */
     routes?: {
         representatives?: Representatives;

package/dist/lib/config.js

@@ -27,6 +27,7 @@ const SHAPE = aShape({
     }),
     analytics: aShape({ linkTracking: aString }),
     features: aShape({ analytics: feature, roadmap: feature }),
+    placeholders: aShape({ sources: anArrayOf(aString), contactEnvKeys: anArrayOf(aString) }),
     routes: aShape({ representatives: aRecordOf(aString) }),
 });
 const CONFIG_FILES = ['officina.config.ts', 'officina.config.mjs', 'officina.config.js'];

package/dist/lib/workflows.d.ts

@@ -1,5 +1,6 @@
 import type { ProjectFiles } from './alignment.ts';
 import type { ContractGap } from './contract.ts';
+export declare const isLive: (line: string) => boolean;
 export declare const mentions: (source: string | undefined, needle: string) => boolean;
 /** Le action del progetto devono stare alla stessa versione del pacchetto installato. */
 export declare function workflowGaps({ read }: ProjectFiles, version: string): ContractGap[];

package/dist/lib/workflows.js

@@ -1,7 +1,7 @@
-// I workflow restano una ricerca testuale — non c'è un parser YAML nel pacchetto — ma una riga
-// commentata non deve soddisfare niente: è il modo più facile per avere un progetto «a posto»
-// che in CI non lancia nessuno di questi passi.
-const liveLines = (source) => source.split(/\r?\n/).filter((line) => !line.trimStart().startsWith('#'));
+// Niente parser YAML nel pacchetto: la ricerca è testuale. Una riga commentata non soddisfa nessun
+// controllo, altrimenti basta commentare un passo per avere un progetto «a posto» che non lo lancia.
+export const isLive = (line) => !line.trimStart().startsWith('#');
+const liveLines = (source) => source.split(/\r?\n/).filter(isLive);
 export const mentions = (source, needle) => source !== undefined && liveLines(source).some((line) => line.includes(needle));
 const ACTIONS = [
     { workflow: '.github/workflows/ci.yml', action: 'ci' },

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@elia97/officina",
-  "version": "0.5.0",
+  "version": "0.6.1",
   "description": "Gli strumenti dei progetti nati dai template di Elia: gate di qualità, generatori e verifiche di build e di produzione.",
   "type": "module",
   "license": "MIT",
@@ -55,7 +55,7 @@
   },
   "devDependencies": {
     "@biomejs/biome": "2.5.10",
-    "@types/node": "^24.13.5",
+    "@types/node": "^26.5.1",
     "@vitest/coverage-v8": "^4.1.11",
     "fallow": "^3.26.0",
     "typescript": "^7.0.2",