@jini-ai/vibecoding 0.4.2 → 0.4.3

package/CHANGELOG.md

@@ -1,5 +1,9 @@
 # Changelog
 
+## 0.4.3 — 2026-10-22
+
+- 2026-10-22: Region parser and session refactors, coverage.
+
 ## 0.4.1
 
 ### Patch Changes

package/dist/html/node/parse5-region-parser.js

@@ -108,14 +108,41 @@ function* walkElements(node) {
     for (const child of children)
         yield* walkElements(child);
 }
-/** Fresh `<div>` fragment-context element — see this module's doc for why `<div>`, not the default. */
-function divFragmentContext() {
-    return defaultTreeAdapter.createElement("div", html.NS.HTML, []);
-}
 /** A parse-error entry, phrased for the model rather than left as parse5's bare kebab-case code. */
 function describeParseError(error) {
     return `${error.code.replace(/-/g, " ")} near character ${error.startOffset}`;
 }
+function analyzeRegion({ element, handle }, _options = {}) {
+    const location = element.sourceCodeLocation;
+    if (location === null || location === undefined || location.startTag === undefined) {
+        return {
+            locationProblem: `a "${AGENT_ELEMENT_ATTRIBUTE}" region ("${handle}") could not be located in the document — ` +
+                "the parser's error-recovery rules likely dropped or repositioned it. Rewrite the document " +
+                "so this element is reachable in ordinary document structure (for example, not a table-only " +
+                "tag used outside a <table>).",
+        };
+    }
+    if (location.endTag === undefined) {
+        return {
+            locationProblem: VOID_ELEMENTS.has(element.tagName)
+                ? `"${handle}" tags a <${element.tagName}> element, which is void and has no inner content — ` +
+                    `void elements cannot be a ${AGENT_ELEMENT_ATTRIBUTE} region`
+                : `a "${AGENT_ELEMENT_ATTRIBUTE}" region ("${handle}") has no closing tag the parser could ` +
+                    "locate — its content boundaries cannot be determined safely. Close the tag explicitly.",
+        };
+    }
+    const role = attributeValue(element, "data-agent-role");
+    const label = attributeValue(element, "data-agent-label");
+    return {
+        region: {
+            handle,
+            innerStart: location.startTag.endOffset,
+            innerEnd: location.endTag.startOffset,
+            ...(role === undefined ? {} : { role }),
+            ...(label === undefined ? {} : { label }),
+        },
+    };
+}
 /**
  * One parse pass, shared by `findRegions` and `checkWellFormed` so both methods judge the same
  * parse5 run rather than parsing the document twice with a chance of disagreeing.
@@ -124,7 +151,9 @@ function describeParseError(error) {
  */
 function analyzeDocument(html_) {
     const parseErrorReasons = [];
-    const fragment = parseFragment(divFragmentContext(), html_, {
+    /** Fresh `<div>` fragment-context element — see this module's doc for why `<div>`, not the default. */
+    const context = defaultTreeAdapter.createElement("div", html.NS.HTML, []);
+    const fragment = parseFragment(context, html_, {
         sourceCodeLocationInfo: true,
         onParseError: (error) => parseErrorReasons.push(describeParseError(error)),
     });
@@ -132,38 +161,14 @@ function analyzeDocument(html_) {
     let locationProblem;
     for (const element of walkElements(fragment)) {
         const handle = attributeValue(element, AGENT_ELEMENT_ATTRIBUTE);
-        if (handle === undefined)
-            continue;
-        if (locationProblem !== undefined)
+        if (handle === undefined || locationProblem !== undefined)
             continue; // Already failed; stop accumulating regions.
-        const location = element.sourceCodeLocation;
-        if (location === null || location === undefined || location.startTag === undefined) {
-            locationProblem =
-                `a "${AGENT_ELEMENT_ATTRIBUTE}" region ("${handle}") could not be located in the document — ` +
-                    "the parser's error-recovery rules likely dropped or repositioned it. Rewrite the document " +
-                    "so this element is reachable in ordinary document structure (for example, not a table-only " +
-                    "tag used outside a <table>).";
+        const analyzed = analyzeRegion({ element, handle });
+        if ("locationProblem" in analyzed) {
+            locationProblem = analyzed.locationProblem;
             continue;
         }
-        if (location.endTag === undefined) {
-            locationProblem = VOID_ELEMENTS.has(element.tagName)
-                ? `"${handle}" tags a <${element.tagName}> element, which is void and has no inner content — ` +
-                    `void elements cannot be a ${AGENT_ELEMENT_ATTRIBUTE} region`
-                : `a "${AGENT_ELEMENT_ATTRIBUTE}" region ("${handle}") has no closing tag the parser could ` +
-                    "locate — its content boundaries cannot be determined safely. Close the tag explicitly.";
-            continue;
-        }
-        regions.push({
-            handle,
-            innerStart: location.startTag.endOffset,
-            innerEnd: location.endTag.startOffset,
-            ...(attributeValue(element, "data-agent-role") === undefined
-                ? {}
-                : { role: attributeValue(element, "data-agent-role") }),
-            ...(attributeValue(element, "data-agent-label") === undefined
-                ? {}
-                : { label: attributeValue(element, "data-agent-label") }),
-        });
+        regions.push(analyzed.region);
     }
     if (locationProblem === undefined) {
         const rawMentions = html_.match(RAW_HANDLE_ATTRIBUTE_RE)?.length ?? 0;

package/dist/react/session.js

@@ -61,6 +61,13 @@ export function createVibecodingSession({ target }, options = {}) {
     function cacheChange(change, side) {
         partContent.set(change.id, side === 'after' ? change.after : change.before);
     }
+    /** Mirrors a recorded snapshot restore in the cache and re-reads the part list it may have grown. */
+    async function restoredPatch(entry) {
+        for (const change of entry.changes)
+            cacheChange(change, 'after');
+        const parts = await target.listParts();
+        return { parts, partContent: new Map(partContent), canUndo: history.canUndo(), canRedo: history.canRedo() };
+    }
     return {
         target,
         history,
@@ -140,20 +147,8 @@ export function createVibecodingSession({ target }, options = {}) {
         takeSnapshot: () => run(async () => ({ result: await target.snapshot(), patch: {} })),
         restoreSnapshot: ({ snapshot }) => run(async () => {
             const entry = await history.restore({ snapshot });
-            if (!entry)
-                return { result: entry, patch: {} };
-            for (const change of entry.changes)
-                cacheChange(change, 'after');
-            const parts = await target.listParts();
-            return {
-                result: entry,
-                patch: {
-                    parts,
-                    partContent: new Map(partContent),
-                    canUndo: history.canUndo(),
-                    canRedo: history.canRedo(),
-                },
-            };
+            // A restore that already matched records no entry and changes nothing to mirror.
+            return { result: entry, patch: entry ? await restoredPatch(entry) : {} };
         }),
     };
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@jini-ai/vibecoding",
-  "version": "0.4.2",
+  "version": "0.4.3",
   "description": "Conversational artifact authoring: a target-agnostic edit loop — addressable parts, validated replace, operation-level undo/redo, and per-turn snapshot/rewind. Framework-free in `/core`; tagged-region HTML documents behind ./html; filesystem targets behind ./node and the chat/preview surface behind ./react.",
   "license": "Apache-2.0",
   "repository": {
@@ -75,10 +75,10 @@
     "typescript": "^5.9.2",
     "vitest": "^3.2.4",
     "parse5": "^8.0.1",
-    "@jini-ai/core": "0.4.2"
+    "@jini-ai/core": "0.4.9"
   },
   "dependencies": {
-    "@jini-ai/core": "^0.4.2"
+    "@jini-ai/core": "^0.4.9"
   },
   "peerDependencies": {
     "react": "^18.3.0 || ^19.0.0",