asciidoclint 1.0.0 → 1.1.0

package/README.md

@@ -122,6 +122,7 @@ extension commands and settings.
 | `AD006/included-document-title` | Included AsciiDoc files should not introduce a level-0 title without level offset |
 | `AD007/heading-depth-limit` | Section headings should not exceed Asciidoctor's supported depth |
 | `AD008/blank-before-list` | Lists should be separated from preceding paragraph text |
+| `AD009/blank-after-list` | Lists should be separated from following structural blocks |
 | `AD010/table-title` | Table blocks should have a title |
 | `AD011/image-title` | Block images should have a title |
 | `AD012/diagram-title` | Diagram blocks should have a title |
@@ -129,14 +130,14 @@ extension commands and settings.
 | `AD016/malformed-figure-caption` | Figure captions should use AsciiDoc title syntax |
 | `AD017/malformed-table-caption` | Table captions should use AsciiDoc title syntax |
 | `AD019/content-after-include` | Text should not be attached directly after include directives |
-| `AD020/appendix-section-level` | Appendices should be section-level blocks in article documents |
+| `AD020/appendix-placement` | Appendix markers should apply to documented appendix section levels |
 | `AD022/circular-include` | Include trees must not contain cycles |
 | `AD023/empty-section` | Sections should contain body content or child sections |
 | `AD024/missing-include` | Include targets should exist after attribute substitution |
 | `AD025/missing-image` | Image targets should exist after attribute substitution |
 | `AD026/missing-xref` | Cross-reference targets should resolve to an anchor or file |
 | `AD027/missing-local-link` | Local link targets should resolve to existing files |
-| `AD028/image-alt-text` | Images should not explicitly set empty alt text |
+| `AD028/image-alt-text` | Images should provide meaningful alt text |
 | `AD029/markdown-link-image-residue` | Markdown link and image residue should not render as text |
 | `AD030/markdown-table-residue` | Markdown pipe table residue should not render as text |
 | `AD031/no-nested-link-text` | Link text should not contain nested links or cross references |
@@ -152,6 +153,20 @@ extension commands and settings.
 | `AD043/section-title-start-left` | Section title syntax should start at the beginning of the line |
 | `AD044/local-adoc-link` | Local AsciiDoc files should be referenced with xref, not link |
 | `AD045/markdown-heading-mix` | Markdown-compatible headings should not be mixed with AsciiDoc headings |
+| `AD046/preface-placement` | Preface sections should match documented book placement |
+| `AD047/abstract-placement` | Abstract sections should match documented article placement |
+| `AD048/bibliography-placement` | Bibliography markers should apply to documented bibliography sections |
+| `AD049/glossary-placement` | Glossary markers should apply to documented glossary section levels |
+| `AD050/index-placement` | Index markers should apply to documented index section levels |
+| `AD051/partintro-placement` | Part introduction markers should apply to the introductory block of a book part |
+| `AD052/acknowledgments-placement` | Acknowledgments markers should apply to documented book section levels |
+| `AD053/dedication-placement` | Dedication markers should apply to documented book section levels |
+| `AD054/colophon-placement` | Colophon markers should apply to documented book section levels |
+| `AD055/docx-anchor-caption-residue` | DOCX anchor caption residue should be converted to AsciiDoc titles |
+| `AD056/dangling-blank-list-continuation` | Blank list continuations should not capture following content |
+| `AD057/semantic-anchor-target` | Semantic anchors should attach to the matching block type |
+| `AD058/docx-bookmark-hash-residue` | DOCX bookmark/hash residue should be cleaned up |
+| `AD059/docx-nested-table-structure` | DOCX-converted nested tables should use valid nested table separators and shallow nesting |
 | `ADW01/unknown-waiver-directive` | Waiver directive names should be known |
 | `ADW02/missing-waiver-rule-list` | Waiver directives should include a rule list |
 | `ADW03/malformed-waiver-rule-list` | Waiver rule lists should use comma-separated rule IDs |
@@ -168,26 +183,30 @@ rules. `ADW##` waiver diagnostics use tags for discovery, but remain always on.
 
 | Group | IDs |
 |---|---|
-| `blank_lines` | `AD008` |
+| `anchor` | `AD057` |
+| `blank_lines` | `AD008`, `AD009` |
 | `blocks` | `AD003`, `AD032`, `AD035` |
 | `accessibility` | `AD028`, `AD042` |
-| `cleanup` | `AD032`, `AD034`, `AD035`, `AD036`, `AD037`, `AD039`, `AD040`, `AD041` |
-| `conversion` | `AD029`, `AD030`, `AD031`, `AD036`, `AD037`, `AD039`, `AD040` |
+| `cleanup` | `AD032`, `AD034`, `AD035`, `AD036`, `AD037`, `AD039`, `AD040`, `AD041`, `AD055`, `AD058`, `AD059` |
+| `conversion` | `AD029`, `AD030`, `AD031`, `AD037`, `AD039`, `AD040`, `AD055`, `AD056`, `AD058`, `AD059` |
+| `docx` | `AD016`, `AD017`, `AD028`, `AD055`, `AD056`, `AD058`, `AD059` |
 | `dependencies` | `AD024`, `AD025`, `AD026`, `AD027` |
 | `diagram` | `AD012` |
 | `format` | `AD041` |
 | `headings` | `AD001`, `AD002`, `AD005`, `AD006`, `AD007`, `AD043`, `AD045` |
-| `image` | `AD011`, `AD013`, `AD016`, `AD025`, `AD028` |
+| `image` | `AD011`, `AD013`, `AD016`, `AD025`, `AD028`, `AD057` |
 | `include` | `AD006`, `AD019`, `AD022`, `AD024` |
 | `inline` | `AD041` |
-| `lists` | `AD008`, `AD036` |
+| `list` | `AD056` |
+| `lists` | `AD008`, `AD009`, `AD036` |
+| `pandoc` | `AD059` |
 | `parser` | `AD000` |
-| `table` | `AD004`, `AD010`, `AD017`, `AD030` |
+| `table` | `AD004`, `AD010`, `AD017`, `AD030`, `AD057`, `AD059` |
 | `waiver` | `ADW01`, `ADW02`, `ADW03`, `ADW04`, `ADW05`, `ADW06`, `ADW07`, `ADW08` |
 | `whitespace` | `AD034` |
 | `references` | `AD023` |
 | `links` | `AD027`, `AD031`, `AD042`, `AD044` |
-| `structure` | `AD043` |
+| `structure` | `AD009`, `AD019`, `AD020`, `AD022`, `AD023`, `AD043`, `AD046`, `AD047`, `AD048`, `AD049`, `AD050`, `AD051`, `AD052`, `AD053`, `AD054`, `AD056`, `AD057` |
 | `markdown-compatibility` | `AD045` |
 | `maintainability` | `AD045` |
 | `xref` | `AD026`, `AD042`, `AD044` |

package/dist/api/lint.js

@@ -23,6 +23,7 @@ async function lintFilesInternal(patterns, options, afterFix) {
         for (const parsedFile of document.files) {
             parsedFiles.set(path.resolve(parsedFile.file), parsedFile);
         }
+        mergeAsciidoctorSections(document, await collectParserSections(file));
         mergeAsciidoctorBlocks(document, await collectParserBlocks(file));
         mergeAsciidoctorReferenceTargets(document, await collectParserReferenceTargets(file));
         resolveDocumentXrefs(document);
@@ -69,24 +70,60 @@ async function collectParserBlocks(file) {
     const { collectAsciidoctorBlocks } = await import("../parsers/asciidoctor.js");
     return collectAsciidoctorBlocks(file);
 }
+async function collectParserSections(file) {
+    const { collectAsciidoctorSections } = await import("../parsers/asciidoctor.js");
+    return collectAsciidoctorSections(file);
+}
 async function collectParserReferenceTargets(file) {
     const { collectAsciidoctorReferenceTargets } = await import("../parsers/asciidoctor.js");
     return collectAsciidoctorReferenceTargets(file);
 }
+function mergeAsciidoctorSections(document, sections) {
+    if (!sections.length) {
+        return;
+    }
+    for (const section of sections) {
+        const existing = document.sections.find((candidate) => (path.resolve(candidate.range.start.file) === path.resolve(section.range.start.file)
+            && candidate.range.start.line === section.range.start.line
+            && candidate.title === section.title));
+        if (!existing) {
+            continue;
+        }
+        existing.sectname = section.sectname;
+        existing.source = "asciidoctor";
+        if (!existing.style && section.style) {
+            existing.style = section.style;
+        }
+    }
+}
 function mergeAsciidoctorBlocks(document, blocks) {
     if (!blocks.length) {
         return;
     }
-    const authoritativeTypes = new Set(blocks.map((block) => block.type));
+    const authoritativeTypes = new Set(blocks
+        .filter((block) => ["table", "image", "diagram"].includes(block.type))
+        .map((block) => block.type));
     const authoritativeFiles = new Set(blocks.map((block) => path.resolve(block.range.start.file)));
-    document.blocks = [
+    const merged = [
         ...document.blocks.filter((block) => (!authoritativeTypes.has(block.type)
             || !authoritativeFiles.has(path.resolve(block.range.start.file)))),
-        ...blocks,
-    ].sort((a, b) => (a.range.start.file.localeCompare(b.range.start.file)
+    ];
+    for (const block of blocks) {
+        if (!merged.some((candidate) => sameBlock(candidate, block))) {
+            merged.push(block);
+        }
+    }
+    document.blocks = merged.sort((a, b) => (a.range.start.file.localeCompare(b.range.start.file)
         || a.range.start.line - b.range.start.line
         || a.range.start.column - b.range.start.column));
 }
+function sameBlock(a, b) {
+    return path.resolve(a.range.start.file) === path.resolve(b.range.start.file)
+        && a.range.start.line === b.range.start.line
+        && a.range.start.column === b.range.start.column
+        && a.type === b.type
+        && a.style === b.style;
+}
 function mergeAsciidoctorReferenceTargets(document, targets) {
     const byKey = new Map();
     for (const target of document.referenceTargets) {

package/dist/api/rules.js

@@ -90,7 +90,7 @@ function configSources(options) {
         return sources;
     }
     const project = findProjectConfig(options.cwd);
-    if (project) {
+    if (project && !sources.some((source) => path.resolve(source.file) === path.resolve(project))) {
         sources.push({ kind: "project", file: project });
     }
     return sources;

package/dist/parsers/asciidoctor.d.ts

@@ -1,4 +1,5 @@
-import type { BlockNode, LintFinding, ReferenceTarget } from "../types.js";
+import type { BlockNode, LintFinding, ReferenceTarget, SectionNode } from "../types.js";
 export declare function collectAsciidoctorDiagnostics(file: string): LintFinding[];
 export declare function collectAsciidoctorBlocks(file: string): BlockNode[];
+export declare function collectAsciidoctorSections(file: string): SectionNode[];
 export declare function collectAsciidoctorReferenceTargets(file: string): ReferenceTarget[];

package/dist/parsers/asciidoctor.js

@@ -2,8 +2,10 @@ import { spawnSync } from "node:child_process";
 import { createRequire } from "node:module";
 import fs from "node:fs";
 import path from "node:path";
+import { specialSectionStyles } from "../rules/specialSections.js";
 const require = createRequire(typeof __filename === "string" ? __filename : import.meta.url);
 let asciidoctor;
+const specialSectionStyleSet = new Set(specialSectionStyles);
 export function collectAsciidoctorDiagnostics(file) {
     if (shouldIsolateAsciidoctor()) {
         return collectAsciidoctorDiagnosticsInChild(file);
@@ -43,6 +45,24 @@ export function collectAsciidoctorBlocks(file) {
         return [];
     }
 }
+export function collectAsciidoctorSections(file) {
+    if (shouldIsolateAsciidoctor()) {
+        return collectAsciidoctorSectionsInChild(file);
+    }
+    try {
+        const processor = getAsciidoctor();
+        const logger = processor.MemoryLogger.create();
+        processor.LoggerManager.setLogger(logger);
+        const document = processor.loadFile(file, {
+            safe: "unsafe",
+            sourcemap: true,
+        });
+        return sectionsFromDocument(document, file);
+    }
+    catch {
+        return [];
+    }
+}
 export function collectAsciidoctorReferenceTargets(file) {
     if (shouldIsolateAsciidoctor()) {
         return collectAsciidoctorReferenceTargetsInChild(file);
@@ -106,6 +126,27 @@ function collectAsciidoctorBlocksInChild(file) {
         return [];
     }
 }
+function collectAsciidoctorSectionsInChild(file) {
+    const asciidoctorEntry = require.resolve("asciidoctor");
+    const child = spawnSync(process.execPath, ["-e", isolatedAsciidoctorSectionsScript(), file, asciidoctorEntry], {
+        encoding: "utf8",
+        env: {
+            ...process.env,
+            ELECTRON_RUN_AS_NODE: "1",
+            ASCIIDOCLINT_ISOLATE_ASCIIDOCTOR: "0",
+        },
+        maxBuffer: 10 * 1024 * 1024,
+    });
+    if (child.status !== 0) {
+        return [];
+    }
+    try {
+        return JSON.parse(child.stdout);
+    }
+    catch {
+        return [];
+    }
+}
 function collectAsciidoctorReferenceTargetsInChild(file) {
     const asciidoctorEntry = require.resolve("asciidoctor");
     const child = spawnSync(process.execPath, ["-e", isolatedAsciidoctorReferenceTargetsScript(), file, asciidoctorEntry], {
@@ -127,6 +168,51 @@ function collectAsciidoctorReferenceTargetsInChild(file) {
         return [];
     }
 }
+function isolatedAsciidoctorSectionsScript() {
+    return `
+const path = require("node:path");
+const file = process.argv[1];
+const asciidoctorEntry = process.argv[2];
+delete globalThis.Opal;
+const asciidoctor = require(asciidoctorEntry)();
+const logger = asciidoctor.MemoryLogger.create();
+asciidoctor.LoggerManager.setLogger(logger);
+function pos(file, line, column) {
+  return { file, line, column };
+}
+function sectionsFromDocument(document) {
+  return (document.findBy({ context: "section" }) || []).flatMap((section) => {
+    const location = section.getSourceLocation && section.getSourceLocation();
+    const sourceFile = location && location.file ? path.resolve(String(location.file)) : path.resolve(file);
+    const line = Number((location && (location.getLineNumber && location.getLineNumber())) || (location && location.lineno) || 1);
+    const title = String((section.getTitle && section.getTitle()) || section.title || "");
+    if (!sourceFile || !line || !title) {
+      return [];
+    }
+    const sectname = (section.getSectionName && section.getSectionName()) || section.sectname;
+    const style = (section.getStyle && section.getStyle()) || undefined;
+    return [{
+      kind: "section",
+      title,
+      level: Number((section.getLevel && section.getLevel()) || section.level || 0),
+      style: style ? String(style) : undefined,
+      sectname: sectname ? String(sectname) : undefined,
+      source: "asciidoctor",
+      range: { start: pos(sourceFile, line, 1) },
+      titleRange: { start: pos(sourceFile, line, 1) },
+      children: [],
+      blocks: [],
+    }];
+  });
+}
+try {
+  const document = asciidoctor.loadFile(file, { safe: "unsafe", sourcemap: true });
+  process.stdout.write(JSON.stringify(sectionsFromDocument(document)));
+} catch {
+  process.stdout.write("[]");
+}
+`;
+}
 function isolatedAsciidoctorScript() {
     return `
 const path = require("node:path");
@@ -157,6 +243,7 @@ process.stdout.write(JSON.stringify(findings));
 `;
 }
 function isolatedAsciidoctorBlocksScript() {
+    const specialStyles = JSON.stringify(specialSectionStyles);
     return `
 const fs = require("node:fs");
 const path = require("node:path");
@@ -166,6 +253,7 @@ delete globalThis.Opal;
 const asciidoctor = require(asciidoctorEntry)();
 const logger = asciidoctor.MemoryLogger.create();
 asciidoctor.LoggerManager.setLogger(logger);
+const specialSectionStyleSet = new Set(${specialStyles});
 function pos(file, line, column) {
   return { file, line, column };
 }
@@ -187,6 +275,12 @@ function blocksFromDocument(document) {
     .concat(document.findBy({ context: "listing" }) || [])
     .concat(document.findBy({ context: "literal" }) || [])
     .filter((block) => diagramStyles.has(String((block.getStyle && block.getStyle()) || "")));
+  const styledNonSections = []
+    .concat(document.findBy((block) => {
+      const context = String((block.getContext && block.getContext()) || "");
+      const style = String((block.getStyle && block.getStyle()) || "");
+      return context !== "section" && specialSectionStyleSet.has(style);
+    }) || []);
   return []
     .concat(tables.map((block) => {
     const location = block.getSourceLocation && block.getSourceLocation();
@@ -197,6 +291,8 @@ function blocksFromDocument(document) {
     return {
       kind: "block",
       type: "table",
+      context: "table",
+      source: "asciidoctor",
       style: block.getStyle && block.getStyle() ? String(block.getStyle()) : undefined,
       title: block.getTitle && block.getTitle() ? String(block.getTitle()) : undefined,
       attributes: Object.fromEntries(Object.entries(attributes).filter(([, value]) => ["string", "number", "boolean"].includes(typeof value)).map(([key, value]) => [key, String(value)])),
@@ -206,7 +302,8 @@ function blocksFromDocument(document) {
     };
   }))
     .concat(images.map((block) => blockNode(block, "image")))
-    .concat(diagrams.map((block) => blockNode(block, "diagram")));
+    .concat(diagrams.map((block) => blockNode(block, "diagram")))
+    .concat(styledNonSections.map((block) => blockNode(block, blockTypeForContext(String((block.getContext && block.getContext()) || "")))));
 }
 function blockNode(block, type) {
   const location = block.getSourceLocation && block.getSourceLocation();
@@ -216,12 +313,17 @@ function blockNode(block, type) {
   return {
     kind: "block",
     type,
+    context: block.getContext && block.getContext() ? String(block.getContext()) : undefined,
+    source: "asciidoctor",
     style: block.getStyle && block.getStyle() ? String(block.getStyle()) : undefined,
     title: block.getTitle && block.getTitle() ? String(block.getTitle()) : undefined,
     attributes: Object.fromEntries(Object.entries(attributes).filter(([, value]) => ["string", "number", "boolean"].includes(typeof value)).map(([key, value]) => [key, typeof value === "boolean" ? value : String(value)])),
     range: { start: pos(sourceFile, startLine, 1) },
   };
 }
+function blockTypeForContext(context) {
+  return ["paragraph","listing","literal","example","sidebar","quote","table","image","admonition","passthrough","stem"].includes(context) ? context : "unknown";
+}
 function primitiveNumber(value) {
   return typeof value === "number" ? value : undefined;
 }
@@ -286,6 +388,11 @@ function blocksFromDocument(document) {
         ...(document.findBy?.({ context: "listing" }) ?? []),
         ...(document.findBy?.({ context: "literal" }) ?? []),
     ].filter((block) => diagramStyles.has(String(block.getStyle?.() ?? "")));
+    const styledNonSections = (document.findBy?.((block) => {
+        const context = String(block.getContext?.() ?? "");
+        const style = String(block.getStyle?.() ?? "");
+        return context !== "section" && specialSectionStyleSet.has(style);
+    }) ?? []);
     return [
         ...tables.map((block) => {
             const location = block.getSourceLocation?.();
@@ -299,6 +406,8 @@ function blocksFromDocument(document) {
             return {
                 kind: "block",
                 type: "table",
+                context: "table",
+                source: "asciidoctor",
                 style: block.getStyle?.() ? String(block.getStyle()) : undefined,
                 title: block.getTitle?.() ? String(block.getTitle()) : undefined,
                 attributes,
@@ -315,8 +424,40 @@ function blocksFromDocument(document) {
         }),
         ...images.map((block) => genericBlockNode(block, "image")),
         ...diagrams.map((block) => genericBlockNode(block, "diagram")),
+        ...styledNonSections.map((block) => genericBlockNode(block, blockTypeForContext(String(block.getContext?.() ?? "")))),
     ].filter((block) => !!block);
 }
+function sectionsFromDocument(document, rootFile) {
+    return (document.findBy?.({ context: "section" }) ?? [])
+        .map((section) => {
+        const location = section.getSourceLocation?.();
+        const sourceFile = location?.file ? path.resolve(String(location.file)) : path.resolve(rootFile);
+        const line = Number(location?.getLineNumber?.() ?? location?.lineno ?? 1);
+        const title = String(section.getTitle?.() ?? section.title ?? "");
+        if (!sourceFile || !line || !title) {
+            return undefined;
+        }
+        const sectname = section.getSectionName?.() ?? section.sectname;
+        const style = section.getStyle?.();
+        return {
+            kind: "section",
+            title,
+            level: Number(section.getLevel?.() ?? section.level ?? 0),
+            style: style ? String(style) : undefined,
+            sectname: sectname ? String(sectname) : undefined,
+            source: "asciidoctor",
+            range: {
+                start: { file: sourceFile, line, column: 1 },
+            },
+            titleRange: {
+                start: { file: sourceFile, line, column: 1 },
+            },
+            children: [],
+            blocks: [],
+        };
+    })
+        .filter((section) => !!section);
+}
 function referenceTargetsFromDocument(document, rootFile) {
     const refs = document.getCatalog?.().refs;
     const map = refs?.$$smap ?? {};
@@ -351,6 +492,8 @@ function genericBlockNode(block, type) {
     return {
         kind: "block",
         type,
+        context: block.getContext?.() ? String(block.getContext()) : undefined,
+        source: "asciidoctor",
         style: block.getStyle?.() ? String(block.getStyle()) : undefined,
         title: block.getTitle?.() ? String(block.getTitle()) : undefined,
         attributes: primitiveAttributes(block.getAttributes?.() ?? {}),
@@ -359,6 +502,12 @@ function genericBlockNode(block, type) {
         },
     };
 }
+function blockTypeForContext(context) {
+    if (["paragraph", "listing", "literal", "example", "sidebar", "quote", "table", "image", "admonition", "passthrough", "stem"].includes(context)) {
+        return context;
+    }
+    return "unknown";
+}
 function tableInfo(block) {
     const rows = block.getRows?.();
     const cells = [

package/dist/parsers/tolerant.js

@@ -29,7 +29,7 @@ export function parseDocument(file) {
         sourceMap: [],
         expandedLine: 1,
     };
-    parseFile(absolute, state, { ...state.attributes });
+    parseFile(absolute, state, { ...state.attributes }, []);
     finalizeXrefDependencies(state);
     const rootLines = state.files.find((entry) => entry.file === absolute)?.lines ?? [];
     const dependencies = { records: state.dependencies };
@@ -48,7 +48,7 @@ export function parseDocument(file) {
         files: state.files,
     };
 }
-function parseFile(file, state, inheritedAttributes) {
+function parseFile(file, state, inheritedAttributes, sectionStack) {
     const absolute = path.resolve(file);
     if (state.visited.has(absolute)) {
         return { ...inheritedAttributes };
@@ -64,7 +64,6 @@ function parseFile(file, state, inheritedAttributes) {
     const lines = text.split(/\r?\n/);
     state.files.push({ file: absolute, lines });
     const attributes = { ...inheritedAttributes };
-    const sectionStack = [];
     let openBlock;
     const conditionalStack = [];
     for (let index = 0; index < lines.length; index += 1) {
@@ -158,7 +157,7 @@ function parseFile(file, state, inheritedAttributes) {
                 status,
             });
             if (status === "resolved" && isAsciiDocSourceFile(resolved)) {
-                Object.assign(attributes, parseFile(resolved, state, attributes));
+                Object.assign(attributes, parseFile(resolved, state, attributes, sectionStack));
             }
             continue;
         }
@@ -220,6 +219,7 @@ function parseFile(file, state, inheritedAttributes) {
                 kind: "section",
                 title,
                 level,
+                style: detectSectionStyle(lines, index),
                 range,
                 titleRange: range,
                 children: [],
@@ -319,7 +319,7 @@ function resolveXref(file, target, state) {
         if (!targetAnchor) {
             return { status: "resolved", resolvedTarget: resolvedFile };
         }
-        parseFile(resolvedFile, state, { ...state.attributes });
+        parseFile(resolvedFile, state, { ...state.attributes }, []);
         return hasAnchor(state, resolvedFile, targetAnchor)
             ? { status: "resolved", resolvedTarget: resolvedFile }
             : { status: "missing" };
@@ -455,6 +455,30 @@ function detectStyle(previousLine) {
     const content = previous.slice(1, -1);
     return content.split(",")[0];
 }
+function detectSectionStyle(lines, headingIndex) {
+    for (let index = headingIndex - 1; index >= 0; index -= 1) {
+        const line = (lines[index] ?? "").trim();
+        if (!line) {
+            return undefined;
+        }
+        if (!line.startsWith("[") || !line.endsWith("]")) {
+            return undefined;
+        }
+        const style = parseStyleName(line.slice(1, -1));
+        if (style) {
+            return style;
+        }
+    }
+    return undefined;
+}
+function parseStyleName(attributeList) {
+    const firstAttribute = attributeList.split(",")[0]?.trim() ?? "";
+    if (!firstAttribute || firstAttribute.startsWith("#") || firstAttribute.startsWith(".") || firstAttribute.startsWith("%")) {
+        return undefined;
+    }
+    const style = firstAttribute.match(/^[^#.%=]+/)?.[0]?.trim();
+    return style || undefined;
+}
 function isProtectedOpenBlock(openBlock) {
     if (!openBlock) {
         return false;

package/dist/rules/AD004.js

@@ -13,6 +13,7 @@ export const AD004 = {
         goodExamples: [{ code: "[cols=\"1,1\"]\n|===\n| one | two\n|===" }],
     },
     function: ({ document }, onError) => {
+        const reportedNestedTables = new Set();
         for (const block of document.blocks.filter((entry) => entry.type === "table")) {
             if (!block.range.end || block.attributes.format || block.attributes.separator) {
                 continue;
@@ -34,6 +35,19 @@ export const AD004 = {
                     fixHelper: "Add the missing cells, remove the extra cells, or make the intended row and column spans explicit.",
                 });
             }
+            for (const finding of findNestedAlternateTableRowIssues(file.lines, block.range.start.line, block.range.end.line)) {
+                const key = `${file.file}:${finding.line}`;
+                if (reportedNestedTables.has(key)) {
+                    continue;
+                }
+                reportedNestedTables.add(key);
+                onError({
+                    severity: "warning",
+                    message: `Nested table row has ${finding.actual} cell${finding.actual === 1 ? "" : "s"} but declares ${finding.expected} columns`,
+                    range: { start: { file: file.file, line: finding.line, column: 1 } },
+                    fixHelper: "Add the missing nested table cell, remove the extra cell marker, or make the intended span explicit so Asciidoctor does not drop cells.",
+                });
+            }
         }
     },
 };
@@ -56,3 +70,88 @@ function containsNestedDefaultPsvTable(lines, startLine, endLine) {
     }
     return false;
 }
+function findNestedAlternateTableRowIssues(lines, startLine, endLine) {
+    const issues = [];
+    for (let index = startLine; index < endLine - 1; index += 1) {
+        const delimiter = parseNestedTableDelimiter(lines[index] ?? "");
+        if (!delimiter || delimiter.marker === "|") {
+            continue;
+        }
+        const expected = findDeclaredColumnCount(lines, index);
+        if (!expected) {
+            continue;
+        }
+        const close = findClosingDelimiter(lines, index + 1, endLine - 1, delimiter.marker);
+        if (close === undefined) {
+            continue;
+        }
+        issues.push(...findIncompleteRows(lines, index + 1, close, delimiter.marker, expected));
+        index = close;
+    }
+    return issues;
+}
+function parseNestedTableDelimiter(line) {
+    const match = line.match(/^\s*([!,/:])={3,}\s*$/);
+    return match ? { marker: match[1] ?? "!" } : undefined;
+}
+function findDeclaredColumnCount(lines, delimiterIndex) {
+    for (let index = delimiterIndex - 1; index >= 0; index -= 1) {
+        const line = lines[index] ?? "";
+        if (line.trim() === "") {
+            continue;
+        }
+        const match = line.match(/^\s*\[.*\bcols\s*=\s*(?:"([^"]+)"|'([^']+)'|([^,\]]+)).*]\s*$/);
+        if (!match) {
+            return undefined;
+        }
+        return parseColumnSpecCount((match[1] ?? match[2] ?? match[3] ?? "").trim());
+    }
+    return undefined;
+}
+function parseColumnSpecCount(value) {
+    const compact = value.match(/^(\d+)\*$/);
+    if (compact) {
+        return Math.max(1, Number(compact[1] ?? "1"));
+    }
+    const numeric = value.match(/^\d+$/);
+    if (numeric) {
+        return Math.max(1, Number(value));
+    }
+    const parts = value.split(",");
+    return Math.max(1, value.includes(",") ? parts.length : parts.filter(Boolean).length);
+}
+function findClosingDelimiter(lines, startIndex, endIndex, marker) {
+    const delimiter = new RegExp(`^\\s*\\${marker}={3,}\\s*$`);
+    for (let index = startIndex; index < endIndex; index += 1) {
+        if (delimiter.test(lines[index] ?? "")) {
+            return index;
+        }
+    }
+    return undefined;
+}
+function findIncompleteRows(lines, startIndex, endIndex, marker, expected) {
+    const issues = [];
+    let rowCells = 0;
+    let rowStartLine;
+    for (let index = startIndex; index < endIndex; index += 1) {
+        const cells = countNestedTableCells(lines[index] ?? "", marker);
+        if (cells === 0) {
+            continue;
+        }
+        rowStartLine ??= index + 1;
+        rowCells += cells;
+        while (rowCells >= expected) {
+            rowCells -= expected;
+            rowStartLine = rowCells > 0 ? index + 1 : undefined;
+        }
+    }
+    if (rowCells > 0) {
+        issues.push({ line: rowStartLine ?? endIndex, expected, actual: rowCells });
+    }
+    return issues;
+}
+function countNestedTableCells(line, marker) {
+    const escapedMarker = marker.replace(/[.*+?^${}()|[\]\\]/g, "\\$&");
+    const markerPattern = new RegExp(`(?<!\\\\)(?:^|\\s)(?:\\d+\\+)?[a-z]?${escapedMarker}`, "g");
+    return [...line.matchAll(markerPattern)].length;
+}

package/dist/rules/AD009.d.ts

@@ -0,0 +1,2 @@
+import type { Rule } from "../types.js";
+export declare const AD009: Rule;