@vesk/adapter 0.2.50-canary.6 → 0.2.50-canary.7

package/dist/seo-audit.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"seo-audit.d.ts","sourceRoot":"","sources":["../src/seo-audit.ts"],"names":[],"mappings":"AAEA,OAAO,KAAK,EAAiB,cAAc,EAAoB,MAAM,yBAAyB,CAAC;AAgI/F,wBAAgB,WAAW,CAAC,MAAM,EAAE,MAAM,EAAE,QAAQ,CAAC,EAAE,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,GAAG,cAAc,CA8C9F"}
+{"version":3,"file":"seo-audit.d.ts","sourceRoot":"","sources":["../src/seo-audit.ts"],"names":[],"mappings":"AAEA,OAAO,KAAK,EAAiB,cAAc,EAAoB,MAAM,yBAAyB,CAAC;AAqK/F,wBAAgB,WAAW,CAAC,MAAM,EAAE,MAAM,EAAE,QAAQ,CAAC,EAAE,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,GAAG,cAAc,CAgD9F"}

package/dist/seo-audit.js

@@ -40,6 +40,16 @@ function collectCombinedSource(appDir) {
     }
     return combined;
 }
+/**
+ * Does this source declare its head as data?
+ *
+ * `export const metadata = defineMetadata({...})` is a complete head on its own,
+ * so every check that asks "is there a title / description / og tag?" has to
+ * consider it — otherwise the audit fails the exact style the docs recommend.
+ */
+function hasMetadataDeclaration(src) {
+    return /\bdefineMetadata\s*\(|export\s+const\s+metadata\b/.test(src);
+}
 const SEO_CHECKS = {
     h1: (src) => {
         const count = (src.match(/<h1[^>]*>/gi) || []).length;
@@ -76,6 +86,16 @@ const SEO_CHECKS = {
         return issues;
     },
     metaDescription: (src) => {
+        if (hasMetadataDeclaration(src)) {
+            // `export const metadata = defineMetadata({ description: … })` is the
+            // declared source of the description; the audit reads source text, so it
+            // can see the declaration but not evaluate it (and must not flag a page
+            // for using the documented API).
+            if (!/\bdescription\s*:/.test(src)) {
+                return [{ severity: SEVERITY.WARN, message: 'No `description` in the metadata declaration' }];
+            }
+            return [];
+        }
         if (!/name=["']description["']/.test(src)) {
             return [{ severity: SEVERITY.WARN, message: 'Missing meta description' }];
         }
@@ -83,6 +103,12 @@ const SEO_CHECKS = {
     },
     ogTags: (src) => {
         const issues = [];
+        if (hasMetadataDeclaration(src)) {
+            // Whether a page wants Open Graph tags is the author's call, and a
+            // metadata declaration is a complete head by itself — so do not invent
+            // three warnings about tags the audit cannot see.
+            return issues;
+        }
         const required = ['og:title', 'og:description', 'og:image'];
         for (const tag of required) {
             const pattern = new RegExp(`property=["']${tag}["']`);
@@ -99,8 +125,18 @@ const SEO_CHECKS = {
         return [];
     },
     title: (src) => {
+        // A static `export const metadata = defineMetadata({ title: … })` is the
+        // documented way to declare a title now, and it produces the same <title>.
+        // Without this, the page style the docs recommend failed `vesk seo
+        // --strict` with "missing <title>".
+        if (hasMetadataDeclaration(src)) {
+            if (!/\btitle\s*:/.test(src)) {
+                return [{ severity: SEVERITY.WARN, message: 'No `title` in the metadata declaration — it will render no <title>' }];
+            }
+            return [];
+        }
         if (!/<title>/i.test(src) && !/<Head>/i.test(src) && !/title>/i.test(src)) {
-            return [{ severity: SEVERITY.ERROR, message: 'Missing <title> or <Head> — page title is critical for SEO' }];
+            return [{ severity: SEVERITY.ERROR, message: 'Missing <title>, <Head>, or `export const metadata` — page title is critical for SEO' }];
         }
         return [];
     },
@@ -125,10 +161,11 @@ export function runSeoAudit(appDir, _options) {
     const combined = collectCombinedSource(appDir);
     if (combined.length === 0) {
         console.error('vesk seo-audit: no page.vsk files found');
-        return { passed: 0, errors: 0, warnings: 0 };
+        return { passed: 0, errors: 0, warnings: 0, issues: [] };
     }
     let errors = 0;
     let warnings = 0;
+    const allIssues = [];
     for (const { path, src } of combined) {
         const relPath = path.replace(appDir, '').replace(/^[\\/]/, '');
         const routeDir = relPath === 'page.vsk' ? '' : relPath.replace(/[\\/]page\.vsk$/, '');
@@ -140,6 +177,7 @@ export function runSeoAudit(appDir, _options) {
         for (const check of Object.values(SEO_CHECKS)) {
             const issues = check(src);
             for (const issue of issues) {
+                allIssues.push({ ...issue, page: relPath || 'page.vsk' });
                 if (issue.severity === SEVERITY.ERROR) {
                     fileErrors++;
                 }
@@ -165,5 +203,5 @@ export function runSeoAudit(appDir, _options) {
     const total = combined.length;
     const status = errors > 0 ? 'FAIL' : (warnings > 0 ? 'PASS_WARN' : 'PASS');
     console.error(`vesk seo-audit: ${total} pages — ${errors} errors, ${warnings} warnings [${status}]`);
-    return { passed: total, errors, warnings };
+    return { passed: total, errors, warnings, issues: allIssues };
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@vesk/adapter",
-  "version": "0.2.50-canary.6",
+  "version": "0.2.50-canary.7",
   "type": "module",
   "description": "Vesk adapter — builds deployable .vesk/ output for Deno and Node serverless",
   "main": "./dist/index.js",
@@ -23,9 +23,9 @@
     "sharp": "^0.35.3"
   },
   "dependencies": {
-    "@vesk/compiler": "^0.2.50-canary.6",
-    "@vesk/runtime": "^0.2.50-canary.6",
-    "@vesk/types": "^0.2.50-canary.6"
+    "@vesk/compiler": "^0.2.50-canary.7",
+    "@vesk/runtime": "^0.2.50-canary.7",
+    "@vesk/types": "^0.2.50-canary.7"
   },
   "devDependencies": {
     "@types/ws": "^8.18.1",