@rafters/veneer 0.0.4 → 0.0.5

package/dist/bin.mjs

@@ -1,6 +1,6 @@
 #!/usr/bin/env node
-import { i as relativeFile } from "./lint-Bqu-nKvP.mjs";
-import { t as check } from "./check-vsAQPYCD.mjs";
+import { i as relativeFile } from "./lint-9HD2n_oJ.mjs";
+import { t as check } from "./check-BWUfShkM.mjs";
 import path from "node:path";
 import { existsSync, realpathSync } from "node:fs";
 import { parseArgs } from "node:util";

package/dist/{check-vsAQPYCD.mjs → check-BWUfShkM.mjs}

@@ -1,4 +1,4 @@
-import { a as resolveComponents, c as readConfig, n as report, o as ProjectError, r as compareCodePoints, s as openProject, t as lint } from "./lint-Bqu-nKvP.mjs";
+import { a as resolveComponents, c as readConfig, n as report, o as ProjectError, r as compareCodePoints, s as openProject, t as lint } from "./lint-9HD2n_oJ.mjs";
 import path from "node:path";
 import { mkdir, rename, rm, writeFile } from "node:fs/promises";
 import { lstatSync } from "node:fs";

package/dist/index.mjs

@@ -1,3 +1,3 @@
-import { n as report, o as ProjectError, s as openProject, t as lint } from "./lint-Bqu-nKvP.mjs";
-import { n as writeDocs, t as check } from "./check-vsAQPYCD.mjs";
+import { n as report, o as ProjectError, s as openProject, t as lint } from "./lint-9HD2n_oJ.mjs";
+import { n as writeDocs, t as check } from "./check-BWUfShkM.mjs";
 export { ProjectError, check, lint, openProject, report, writeDocs };

package/dist/{lint-Bqu-nKvP.mjs → lint-9HD2n_oJ.mjs}

@@ -991,6 +991,17 @@ function valueAttributes({ ts }, sourceFile) {
 	visit(sourceFile);
 	return found;
 }
+const FREEHAND_HELP = "use Container or Grid for layout";
+function openingElements({ ts }, sourceFile) {
+	const found = [];
+	const visit = (node) => {
+		if (ts.isJsxOpeningLikeElement(node)) found.push(node);
+		ts.forEachChild(node, visit);
+	};
+	visit(sourceFile);
+	return found;
+}
+const isDiv = (ts, { tagName }) => ts.isIdentifier(tagName) && tagName.text === "div";
 async function lint(project, options) {
 	const files = options.files?.filter((file) => !file.endsWith(".ts") || isClassesFile(file)) ?? project.checker.getRootFileNames().filter((file) => file.endsWith(".tsx") || file.endsWith(".astro") || isClassesFile(file)).map((file) => relativeFile(project.root, file));
 	const perFile = [];
@@ -1104,6 +1115,33 @@ async function lint(project, options) {
 		}
 		return flagged;
 	};
+	const isComponentUse = (element) => {
+		if (isDiv(ts, element)) return false;
+		const component = resolveTag(reader, element);
+		const declaration = component?.valueDeclaration ?? component?.declarations?.[0];
+		return declaration !== void 0 && isCheckedUse(declaration.getSourceFile());
+	};
+	const freehandDivs = (file, sourceFile) => {
+		const elements = openingElements(reader, sourceFile);
+		const divs = elements.filter((element) => isDiv(ts, element));
+		if (divs.length === 0 || !elements.some(isComponentUse)) return [];
+		return divs.flatMap(({ tagName }) => {
+			const start = positionOf(reader, sourceFile, tagName.getStart(sourceFile));
+			const end = positionOf(reader, sourceFile, tagName.end);
+			if (!start || !end) return [];
+			return [{
+				file,
+				line: start.line,
+				column: start.column,
+				endLine: end.line,
+				endColumn: end.column,
+				severity: "warning",
+				code: "rafters(freehand-div)",
+				message: "a div laid by hand around rafters components",
+				help: FREEHAND_HELP
+			}];
+		});
+	};
 	const found = [];
 	for (const [file, diagnostics] of perFile) {
 		const absolute = path.join(project.root, file);
@@ -1116,17 +1154,18 @@ async function lint(project, options) {
 		if (!isExempt(sourceFile)) {
 			found.push(...bracketValues(reader, file, sourceFile, isOwnSource));
 			found.push(...await badValues(file, sourceFile));
+			found.push(...freehandDivs(file, sourceFile));
 		}
 	}
 	return found.sort((a, b) => compareCodePoints(a.file, b.file) || a.line - b.line || a.column - b.column);
 }
-function firstLine(message) {
-	return (typeof message === "string" ? message : message.value).split(/\r\n|\r|\n/, 1)[0] ?? "";
+function messageLines(message) {
+	return (typeof message === "string" ? message : message.value).split(/\r\n|\r|\n/);
 }
 function report(diagnostics) {
 	const lines = diagnostics.flatMap(({ file, line, column, severity, code, message, help }) => {
-		const entry = `${file}:${line}:${column} ${severity} ${code} ${firstLine(message)}`;
-		return help === void 0 ? [entry] : [entry, `  help: ${help}`];
+		const entry = messageLines(message).map((text, index) => index === 0 ? `${file}:${line}:${column} ${severity} ${code} ${text}` : `  ${text}`);
+		return help === void 0 ? entry : [...entry, `  help: ${help}`];
 	});
 	const count = diagnostics.length;
 	lines.push(count === 1 ? "1 problem" : `${count} problems`);

package/dist/worker.mjs

@@ -1,4 +1,4 @@
-import { c as readConfig, s as openProject, t as lint } from "./lint-Bqu-nKvP.mjs";
+import { c as readConfig, s as openProject, t as lint } from "./lint-9HD2n_oJ.mjs";
 import path from "node:path";
 import { existsSync } from "node:fs";
 import { promisify } from "node:util";

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@rafters/veneer",
-  "version": "0.0.4",
+  "version": "0.0.5",
   "description": "Docs, rafters check and the composite compile for rafters projects",
   "license": "MIT",
   "repository": {