spec-up-t 1.4.0 → 1.4.1-beta.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "spec-up-t",
-  "version": "1.4.0",
+  "version": "1.4.1-beta.1",
   "description": "Technical specification drafting tool that generates rich specification documents from markdown. Forked from https://github.com/decentralized-identity/spec-up by Daniel Buchner (https://github.com/csuwildcat)",
   "main": "./index",
   "repository": {

package/src/pipeline/references/fetch-terms-from-index.js

@@ -105,17 +105,15 @@ async function fetchAllTermsFromIndex(token, owner, repo, options = {}) {
                 return;
             }
 
-            let termText = '';
-            termSpan.childNodes.forEach(node => {
-                if (node.nodeType === dom.window.Node.TEXT_NODE) {
-                    termText += node.textContent.trim();
-                }
-            });
-
-            if (!termText) {
-                termText = termSpan.textContent.trim();
+            // Extract the canonical term identifier from the term-local-original-term span.
+            // This contains the original term identifier as used in the source file naming
+            // and tref/xref references. If this span doesn't exist, skip the term entirely.
+            const originalTermSpan = dt.querySelector('span.term-local-original-term');
+            if (!originalTermSpan) {
+                return;
             }
 
+            const termText = originalTermSpan.textContent.trim();
             if (!termText) {
                 return;
             }

package/src/pipeline/references/process-xtrefs-data.js

@@ -14,7 +14,7 @@ async function processXTrefsData(allXTrefs, GITHUB_API_TOKEN, outputPathJSON, ou
 
         allXTrefs.xtrefs = allXTrefs.xtrefs.filter(xtref => {
             if (!xtref.owner || !xtref.repo || !xtref.repoUrl) {
-                Logger.warn(`Removing incomplete reference: ${xtref.externalSpec}, ${xtref.term}`);
+                Logger.error(`Removing incomplete reference: ${xtref.externalSpec}, ${xtref.term}`);
                 return false;
             }
             return true;

package/src/utils/git-info.js

@@ -25,7 +25,7 @@ function getCurrentBranch() {
         }).trim();
         
         if (branch) {
-            Logger.info(`Current git branch: ${branch}`);
+            // This is too much info, comment out for the moment, Logger.info(`Current git branch: ${branch}`);
             return branch;
         }
         
@@ -61,7 +61,6 @@ function getGithubRepoInfo(spec) {
         const account = source.account || 'unknown';
         const repo = source.repo || 'unknown';
         const branch = getCurrentBranch();
-        
         const content = `${account},${repo},${branch}`;
         Logger.info(`GitHub repo info: ${content}`);
         return content;

package/src/pipeline/references/match-term.js

@@ -1,34 +0,0 @@
-/**
- * @file match-term.js
- * @description Utilities for matching a specific term within a [[def: ...]] definition block.
- */
-
-const { isLineWithDefinition } = require('../../utils/is-line-with-definition');
-const Logger = require('../../utils/logger');
-
-function matchTerm(text, term) {
-    if (!text || typeof text !== 'string') {
-        Logger.warn('Nothing to match for term:', term);
-        return false;
-    }
-
-    const firstLine = text.split('\n')[0].trim();
-    if (!isLineWithDefinition(firstLine)) {
-        Logger.warn('String does not start with `[[def:` or end with `]]`');
-        return false;
-    }
-
-    const startPos = firstLine.indexOf('[[def:') + 6;
-    const endPos = firstLine.indexOf(']]');
-    if (startPos === -1 || endPos === -1) {
-        return false;
-    }
-
-    const relevantPart = firstLine.substring(startPos, endPos);
-    const termsArray = relevantPart.split(',').map(entry => entry.trim());
-    return termsArray.includes(term);
-}
-
-module.exports = {
-    matchTerm
-};

package/src/pipeline/references/match-term.test.js

@@ -1,36 +0,0 @@
-const { matchTerm } = require('./match-term');
-
-// Tests for matching terms within definition markup
-describe('matchTerm', () => {
-    // Test: Can the system find terms that exist in definition markup?
-    test('returns true when the term is found in a correctly formatted definition', () => {
-        const text = '[[def: term1, term2]]\nSome additional text';
-        expect(matchTerm(text, 'term1')).toBe(true);
-        expect(matchTerm(text, 'term2')).toBe(true);
-    });
-
-    // Test: Does the system correctly identify when a term is not present?
-    test('returns false when the term is not found in the definition', () => {
-        const text = '[[def: term1, term2]]\nSome additional text';
-        expect(matchTerm(text, 'term3')).toBe(false);
-    });
-
-    // Test: Does the system handle invalid input gracefully?
-    test('returns false when the text is null or not a string', () => {
-        expect(matchTerm(null, 'term1')).toBe(false);
-        expect(matchTerm(123, 'term1')).toBe(false);
-    });
-
-    // Test: Does the system require proper definition markup format?
-    test('returns false when the first line is not a valid definition', () => {
-        const text = 'Invalid definition line\n[[def: term1, term2]]';
-        expect(matchTerm(text, 'term1')).toBe(false);
-    });
-
-    // Test: Can the system handle whitespace variations in markup?
-    test('handles extra spaces correctly', () => {
-        const text = '[[def:   term1  ,   term2   ]]';
-        expect(matchTerm(text, 'term1')).toBe(true);
-        expect(matchTerm(text, 'term2')).toBe(true);
-    });
-});