ucn 5.3.8 → 5.4.1

package/core/output/endpoints.js

@@ -55,7 +55,7 @@ function formatUncertainRequests(result, options) {
     lines.push(`Possible client requests (${list.length}) — request-shaped call with a path literal, receiver not recognized as an HTTP client:`);
     const cap = options.all ? Infinity : 10;
     for (const r of list.slice(0, cap)) {
-        lines.push(`  ${r.file}:${r.line} ${r.receiver}.${r.method}(${JSON.stringify(r.path)}) in ${r.callerName}`);
+        lines.push(`  ${r.file}:${r.line} ${r.receiver}.${r.method}(${JSON.stringify(r.path)}) in ${r.callerName}${r.isTest ? ' [test]' : ''}`);
     }
     if (list.length > cap) lines.push(`  (+${list.length - cap} more — use --all)`);
     return lines;
@@ -88,7 +88,7 @@ function formatRoutesAndRequests(routes, requests, meta, options, advisory = nul
                 for (const r of list) {
                     const handler = r.handler || '<anonymous>';
                     const fw = r.framework ? `[${r.framework}]` : '';
-                    lines.push(`  ${pad(r.method, 7)} ${pad(r.path, 40)} → ${handler} ${fw} :${r.line}`);
+                    lines.push(`  ${pad(r.method, 7)} ${pad(r.path, 40)} → ${handler} ${fw}${r.isTest ? ' [test]' : ''} :${r.line}`);
                 }
                 lines.push('');
             }
@@ -115,7 +115,7 @@ function formatRoutesAndRequests(routes, requests, meta, options, advisory = nul
                     const inferred = r.methodInferred ? '?' : '';
                     const interp = r.interp ? ' (interp)' : '';
                     const fw = r.framework ? `[${r.framework}]` : '';
-                    lines.push(`  ${pad(r.method + inferred, 7)} ${pad(r.path + interp, 40)} from ${r.callerName} ${fw} :${r.line}`);
+                    lines.push(`  ${pad(r.method + inferred, 7)} ${pad(r.path + interp, 40)} from ${r.callerName} ${fw}${r.isTest ? ' [test]' : ''} :${r.line}`);
                 }
                 lines.push('');
             }
@@ -166,12 +166,12 @@ function formatBridges(bridges, unmatchedRoutes, unmatchedRequests, meta, option
 
         lines.push(`Matched (${sorted.length} routes):`);
         for (const { route, clients } of sorted) {
-            lines.push(`  ${pad(route.method, 7)} ${route.path}  [${route.framework}]  ${route.file}:${route.line}`);
+            lines.push(`  ${pad(route.method, 7)} ${route.path}  [${route.framework}]${route.isTest ? ' [test]' : ''}  ${route.file}:${route.line}`);
             for (const b of clients) {
                 const conf = b.confidence.toFixed(2);
                 const tier = b.matchType.toUpperCase();
                 const inf = b.methodInferred ? ' method?' : '';
-                lines.push(`    ↔ ${pad(b.request.method + inf, 9)} ${pad(b.request.path, 30)}  ${tier} (${conf})  from ${b.request.callerName}  ${b.request.file}:${b.request.line}`);
+                lines.push(`    ↔ ${pad(b.request.method + inf, 9)} ${pad(b.request.path, 30)}  ${tier} (${conf})  from ${b.request.callerName}${b.request.isTest ? ' [test]' : ''}  ${b.request.file}:${b.request.line}`);
             }
             lines.push('');
         }
@@ -180,7 +180,7 @@ function formatBridges(bridges, unmatchedRoutes, unmatchedRequests, meta, option
     if (unmatchedRoutes.length > 0) {
         lines.push(`Unmatched server routes (${unmatchedRoutes.length}):`);
         for (const r of unmatchedRoutes) {
-            lines.push(`  ${pad(r.method, 7)} ${pad(r.path, 40)} → ${r.handler}  [${r.framework}]  ${r.file}:${r.line}`);
+            lines.push(`  ${pad(r.method, 7)} ${pad(r.path, 40)} → ${r.handler}  [${r.framework}]${r.isTest ? ' [test]' : ''}  ${r.file}:${r.line}`);
         }
         lines.push('');
     }
@@ -190,7 +190,7 @@ function formatBridges(bridges, unmatchedRoutes, unmatchedRequests, meta, option
         for (const r of unmatchedRequests) {
             const inferred = r.methodInferred ? '?' : '';
             const interp = r.interp ? ' (interp)' : '';
-            lines.push(`  ${pad(r.method + inferred, 7)} ${pad(r.path + interp, 40)} from ${r.callerName}  [${r.framework}]  ${r.file}:${r.line}`);
+            lines.push(`  ${pad(r.method + inferred, 7)} ${pad(r.path + interp, 40)} from ${r.callerName}  [${r.framework}]${r.isTest ? ' [test]' : ''}  ${r.file}:${r.line}`);
         }
     }
 
@@ -213,6 +213,7 @@ function formatEndpointsJson(result, options = {}) {
         file: r.file,
         line: r.line,
         framework: r.framework,
+        isTest: !!r.isTest,
         ...(r.classPrefix && { classPrefix: r.classPrefix }),
     });
     const trimReq = (r) => ({
@@ -225,6 +226,7 @@ function formatEndpointsJson(result, options = {}) {
         callerName: r.callerName,
         ...(r.callerStartLine && { callerStartLine: r.callerStartLine }),
         framework: r.framework,
+        isTest: !!r.isTest,
         ...(r.methodInferred && { methodInferred: true }),
     });
     const trimBridge = (b) => ({
@@ -250,6 +252,7 @@ function formatEndpointsJson(result, options = {}) {
             uncertainRequests: (result.uncertainRequests || []).map(r => ({
                 receiver: r.receiver, method: r.method, path: r.path,
                 file: r.file, line: r.line, callerName: r.callerName, reason: r.reason,
+                isTest: !!r.isTest,
             })),
             // In unmatched-only mode, the matched bridges array is suppressed
             // — consumers that want both should not pass --unmatched.

package/core/output/lines.js

@@ -77,9 +77,20 @@ function findRecords(result) {
 function usagesRecords(result) {
     const out = [];
     const notes = [];
+    const byLine = new Map();
     for (const usage of Array.isArray(result) ? result : []) {
         const kind = usage.isDefinition ? 'definition' : (usage.usageType || 'reference');
-        out.push(record(pathOf(usage), usage.line, usage.content, kind === 'call' ? '' : kind));
+        const key = `${pathOf(usage)}\0${usage.line}`;
+        if (!byLine.has(key)) byLine.set(key, { usage, kinds: new Set(), count: 0 });
+        const row = byLine.get(key);
+        row.kinds.add(kind);
+        row.count++;
+    }
+    for (const { usage, kinds, count } of byLine.values()) {
+        const tags = [...kinds].filter(kind => kind !== 'call');
+        if (kinds.has('call') && tags.length) tags.unshift('call');
+        if (count > 1) tags.push(`${count} occurrences`);
+        out.push(record(pathOf(usage), usage.line, usage.content, tags.join('; ')));
     }
     const counts = result && result.summaryCounts;
     if (counts && counts.hiddenTestUsages > 0) {
@@ -187,6 +198,17 @@ function impactRecords(result) {
                 out.push(record(pathOf(site), site.line, site.content, 'unverified: deleted-target-name-match'));
             }
         }
+        for (const symbol of [...(result.symbols || []), ...(result.newSymbols || []), ...(result.deletedSymbols || [])]) {
+            out.push(record(symbol.relativePath, symbol.startLine, symbol.name, `${symbol.type} declaration change`));
+            if (symbol.impact) {
+                const nested = impactRecords(symbol.impact);
+                out.push(...nested.records);
+                notes.push(...nested.notes);
+            }
+            for (const site of symbol.remainingReferences || []) {
+                out.push(record(site.file, site.line, site.expression, 'unverified: deleted-target-name-match'));
+            }
+        }
         const summary = result.summary || {};
         notes.push(`# Diff: ${summary.modifiedFunctions || 0} modified, ${summary.newFunctions || 0} new, ${summary.deletedFunctions || 0} deleted functions; ${(result.moduleLevelChanges || []).length} file(s) with module-level changes.`);
         if (result.nonSourcePaths) notes.push(`# ${result.nonSourcePaths} changed path(s) outside supported source files not analyzed.`);
@@ -206,11 +228,11 @@ function impactRecords(result) {
         const accesses = result.propertyAccesses;
         for (const group of accesses.byFile || []) {
             for (const access of group.sites || []) {
-                out.push(record(group.file, access.line, access.expression, 'property-access'));
+                out.push(record(group.file, access.line, access.expression, `property-access: ${access.accessKind || 'access'}${Number.isInteger(access.column) ? `, column ${access.column + 1}` : ''}`));
             }
         }
         for (const access of accesses.unverifiedSites || []) {
-            out.push(record(pathOf(access), access.line, access.expression, `${unverifiedTag(access)}; property-access`));
+            out.push(record(pathOf(access), access.line, access.expression, `${unverifiedTag(access)}; property-access: ${access.accessKind || 'access'}${Number.isInteger(access.column) ? `, column ${access.column + 1}` : ''}`));
         }
         notes.push(`# PROPERTY ACCESS SITES: ${accesses.confirmedCount} confirmed, ${accesses.unverifiedCount} unverified, ${accesses.excluded?.total || 0} other-target (separate from caller ACCOUNT).`);
     }

package/core/output/public.js

@@ -40,12 +40,20 @@ function appendNote(text, note) {
 }
 
 /** Canonicalize object keys so JSON bytes do not depend on index provenance. */
-function canonicalJsonValue(value) {
-    if (Array.isArray(value)) return value.map(canonicalJsonValue);
+function canonicalJsonValue(value, root, field = undefined) {
+    if (Array.isArray(value)) return value.map(item => canonicalJsonValue(item, root, field));
+    if (root && typeof value === 'string' &&
+        ['file', 'filePath', 'callerFile', 'definitionFile', 'resolved', 'path', 'targetFile', 'from', 'to', 'root', 'files'].includes(field)) {
+        const path = require('path');
+        if (path.isAbsolute(value)) {
+            const relative = path.relative(root, value);
+            if (relative && relative !== '..' && !relative.startsWith('..' + path.sep) && !path.isAbsolute(relative)) return relative;
+        }
+    }
     if (!value || typeof value !== 'object') return value;
     const canonical = {};
     for (const key of Object.keys(value).sort()) {
-        canonical[key] = canonicalJsonValue(provenanceReplacer(key, value[key]));
+        canonical[key] = canonicalJsonValue(provenanceReplacer(key, value[key]), root, key);
     }
     return canonical;
 }
@@ -474,12 +482,13 @@ function formatPublicJson(command, result, params = {}, execution = {}) {
             ...(data && data.ok === false && { ok: false }),
             ...(modeOf(command, result) && { mode: modeOf(command, result) }),
             contract: contractMeta(command),
+            ...(execution.projectRoot && { pathBase: execution.projectRoot }),
             ...commandMeta,
             ...(execution.note && { note: execution.note }),
         },
         data,
     };
-    return JSON.stringify(canonicalJsonValue(envelope), null, 2);
+    return JSON.stringify(canonicalJsonValue(envelope, execution.projectRoot), null, 2);
 }
 
 module.exports = {

package/core/output/refactoring.js

@@ -522,7 +522,10 @@ function formatAuditAsync(result) {
         lines.push(`${file} (${fileIssues.length})`);
         for (const issue of fileIssues) {
             const caller = issue.callerName ? ` [${issue.callerName}]` : '';
-            lines.push(`  :${issue.line}${caller}  ${issue.calleeName}() — async, not awaited`);
+            const detail = issue.reason === 'stored-promise-used-as-value'
+                ? `${issue.variable} used as a resolved value; promise from ${issue.calleeName}() at line ${issue.originLine}`
+                : `${issue.calleeName}() — async, not awaited`;
+            lines.push(`  :${issue.line}${caller}  ${detail}`);
         }
     }
     return lines.join('\n');

package/core/output/reporting.js

@@ -284,7 +284,7 @@ function formatDeadcode(results, options = {}) {
         lines.push(`\n${extHint}`);
     }
     if (results.excludedRuntimeContract > 0) {
-        lines.push(`\n${results.excludedRuntimeContract} Java serialization callback(s) hidden (JVM runtime contract, not dead).`);
+        lines.push(`\n${results.excludedRuntimeContract} runtime callback(s) hidden (language/runtime registrations, not dead).`);
     }
     if (results.pythonImplicitExportFiles > 0) {
         lines.push(`\nPython public-surface rule active in ${results.pythonImplicitExportFiles} file(s) without __all__: top-level non-underscore names are treated as externally reachable.`);

package/core/output-budget.js

@@ -132,6 +132,7 @@ function applyOutputBudget(text, {
     all = false,
     surface = 'cli',
     params = {},
+    trailingChars = 0,
 } = {}) {
     if (!text) {
         return {
@@ -148,13 +149,14 @@ function applyOutputBudget(text, {
         ? BROAD_OUTPUT_CHARS
         : DEFAULT_OUTPUT_CHARS;
     const requested = maxChars || (all ? MAX_OUTPUT_CHARS : defaultLimit);
-    const limit = Math.min(requested, MAX_OUTPUT_CHARS);
+    const hardLimit = Math.min(requested, MAX_OUTPUT_CHARS);
+    const limit = Math.max(0, hardLimit - trailingChars);
     if (text.length <= limit) {
         return {
             text,
             truncated: false,
             fullChars: text.length,
-            requestedLimit: limit,
+            requestedLimit: hardLimit,
             contractMetadata: [],
             contractMetadataComplete: true,
         };
@@ -178,8 +180,8 @@ function applyOutputBudget(text, {
         ? `Raise ${raiseHint}.`
         : `Narrow with ${compactScope || raiseHint}; raise ${raiseHint}.`;
     let notice = compactBudget
-        ? `... OUTPUT TRUNCATED (${text.length}→${limit}). ${compactGuidance}`
-        : `... OUTPUT TRUNCATED: ${text.length} chars total; hard limit ${limit}. ` +
+        ? `... OUTPUT TRUNCATED (${text.length}→${hardLimit}). ${compactGuidance}`
+        : `... OUTPUT TRUNCATED: ${text.length} chars total; hard limit ${hardLimit}. ` +
             `${narrowingHint(command, surface, params)} ${allHint}`;
     if (compactBudget && notice.length > limit) {
         const emergency = supportsAll
@@ -225,7 +227,7 @@ function applyOutputBudget(text, {
             text: pieces.join('\n').slice(0, limit),
             truncated: true,
             fullChars: text.length,
-            requestedLimit: limit,
+            requestedLimit: hardLimit,
             contractMetadata: contractMetadata.lines,
             contractMetadataComplete: contractMetadata.complete,
         };
@@ -283,7 +285,7 @@ function applyOutputBudget(text, {
         text: rendered,
         truncated: true,
         fullChars: text.length,
-        requestedLimit: limit,
+        requestedLimit: hardLimit,
         contractMetadata: contractMetadata.lines,
         contractMetadataComplete: contractMetadata.complete,
     };

package/core/project.js

@@ -1105,6 +1105,11 @@ class ProjectIndex {
             const lowerPath = filePath.toLowerCase();
             for (const pattern of filters.exclude) {
                 const lowerPattern = pattern.toLowerCase();
+                if (lowerPattern === 'test files') {
+                    const rp = path.isAbsolute(filePath) ? path.relative(this.root, filePath) : filePath;
+                    if (require('./shared').isTestPath(rp)) return false;
+                    continue;
+                }
                 let regex = this._excludeRegexCache?.get(lowerPattern);
                 if (!regex) {
                     const escaped = lowerPattern.replace(/[.*+?^${}()|[\]\\]/g, '\\$&');

package/core/registry.js

@@ -149,7 +149,7 @@ const FLAG_APPLICABILITY = {
     repo:         ['file', 'exclude', 'top', 'limit', 'all', 'detailed', 'topLevel', 'in', 'functions', 'hot', 'deep', 'sections'],
     deadcode:     ['file', 'exclude', 'includeTests', 'includeExported', 'includeDecorated', 'limit', 'in'],
     entrypoints:  ['file', 'exclude', 'includeTests', 'excludeTests', 'limit', 'type', 'framework'],
-    endpoints:    ['file', 'exclude', 'limit', 'framework', 'bridge', 'serverOnly', 'clientOnly', 'unmatched', 'method', 'prefix', 'hideUncertain'],
+    endpoints:    ['file', 'in', 'exclude', 'excludeTests', 'limit', 'framework', 'bridge', 'serverOnly', 'clientOnly', 'unmatched', 'method', 'prefix', 'hideUncertain'],
     stacktrace:   ['stack'],
     auditAsync:   ['file', 'exclude', 'limit'],
 };

package/core/search.js

@@ -727,6 +727,7 @@ function structuralSearch(index, options = {}) {
         // Auto-infer type: --receiver implies type=call
         const type = options.type || (receiver ? 'call' : undefined);
         const results = [];
+        const skippedTestFiles = new Set();
 
         // Validate type if provided
         if (type && !STRUCTURAL_TYPES.has(type)) {
@@ -753,7 +754,13 @@ function structuralSearch(index, options = {}) {
                 if (!rp.includes(options.file) && !rp.endsWith(options.file)) return false;
             }
             if ((options.exclude && options.exclude.length > 0) || options.in) {
-                if (!index.matchesFilters(fileEntry.relativePath, { exclude: options.exclude, in: options.in })) return false;
+                if (!index.matchesFilters(fileEntry.relativePath, { in: options.in })) return false;
+                if (!index.matchesFilters(fileEntry.relativePath, { exclude: options.exclude })) {
+                    if (options.testExclude && !index.matchesFilters(fileEntry.relativePath, { exclude: options.testExclude })) {
+                        skippedTestFiles.add(fileEntry.relativePath);
+                    }
+                    return false;
+                }
             }
             return true;
         };
@@ -978,6 +985,7 @@ function structuralSearch(index, options = {}) {
                 }).filter(([, v]) => v !== undefined && v !== null)),
                 totalMatched: total,
                 shown: results.length,
+                filesSkipped: skippedTestFiles.size,
                 ...(unused && {
                     unusedScope: 'callable-symbols-only',
                     unusedSafety: 'candidate-only; use deadcode before deletion',

package/core/shared.js

@@ -50,15 +50,22 @@ function codeUnitCompare(a, b) {
  * Path-based test heuristic — matches the same patterns as `find`'s exclusion
  * logic so that `about` and `find` agree on which files are de-emphasized.
  *
- * Triggers when any of `test|tests|spec|__tests__|__mocks__|fixture|mock`
- * appears as a path segment (with word boundaries on both sides).
+ * Uses language-specific filename conventions plus `test|tests|__tests__|__mocks__|fixture|mock`
+ * as path segments (with word boundaries on both sides). `spec` is a test
+ * marker only for languages whose test-file conventions include it.
  *
  * Complement to `isTestFile` (filename pattern check) — together they catch
  * both `foo.test.js` (filename) AND `test/agent-benchmark.js` (directory).
  */
 function isTestPath(rp) {
     if (!rp) return false;
-    return /(^|[/._-])(test|tests|spec|__tests__|__mocks__|fixture|mock)s?([/._-]|$)/i.test(rp);
+    const language = detectLanguage(rp);
+    const { langTraits } = require('../languages');
+    const specConvention = langTraits(language)?.testFileCandidates?.('sample', '')
+        .some(candidate => candidate.includes('.spec'));
+    return isTestFile(rp, language) ||
+        /(^|[/._-])(test|tests|__tests__|__mocks__|fixture|mock)s?([/._-]|$)/i.test(rp) ||
+        (!!specConvention && /(^|[/._-])specs?([/._-]|$)/i.test(rp));
 }
 
 /**
@@ -109,7 +116,9 @@ function pickBestDefinition(matches, opts = {}) {
  * Returns a new array with test patterns appended (deduplicating).
  */
 function addTestExclusions(exclude) {
-    const testPatterns = ['test', 'spec', '__tests__', '__mocks__', 'fixture', 'mock'];
+    // Keep automatic test classification distinct from explicit substring
+    // exclusions: --exclude=spec deliberately excludes Python specs too.
+    const testPatterns = ['test files'];
     const existing = new Set((exclude || []).map(e => e.toLowerCase()));
     const additions = testPatterns.filter(p => !existing.has(p));
     return [...(exclude || []), ...additions];

package/core/verify.js

@@ -247,14 +247,26 @@ function classifyCallContext(callNode, language) {
 /**
  * Find a call expression node at the target line matching funcName
  */
-function findCallNode(node, callTypes, targetRow, funcName, occurrence = 0) {
+function findCallNode(node, callTypes, targetRow, funcName, occurrence = 0, site = null) {
     // Several same-name calls can share one line (`greet("a") + greet("b")`,
     // f-strings) — fix #231: callers pass the site's per-line ordinal so each
     // record is arg-checked against ITS OWN node, not the line's first.
     // Records and this walk are both pre-order, so ordinals align; an
     // out-of-range ordinal falls back to the first match (never worse than
     // the pre-fix behavior when a parse shape hides a node).
-    const matches = _collectCallNodes(node, callTypes, targetRow, funcName, occurrence + 1);
+    const matches = _collectCallNodes(node, callTypes, targetRow, funcName, site ? Infinity : occurrence + 1);
+    if (Number.isInteger(site?.start)) {
+        // The resolver already identified the callee token. Preserve that
+        // identity through argument extraction, including aliases and several
+        // calls on one line (some of which may target another declaration).
+        const exact = matches.filter(candidate => {
+            const callee = candidate.childForFieldName('function') ||
+                candidate.childForFieldName('name') ||
+                candidate.childForFieldName('constructor') || candidate.childForFieldName('type');
+            return callee && callee.startIndex <= site.start && callee.endIndex >= site.end;
+        });
+        if (exact.length === 1) return exact[0];
+    }
     return matches[occurrence] || matches[0] || null;
 }
 
@@ -795,8 +807,10 @@ function computePlanCallSites(index, name, def) {
             content: c.content,
             usageType: 'call',
             receiver: c.receiver,
+            calledAs: c.calledAs,
+            callSite: c.provenance?.facts?.site,
         };
-        const siteKey = `${c.file}:${c.line}`;
+        const siteKey = `${c.file}:${c.line}:${c.calledAs || name}`;
         const occurrence = planLineSeen.get(siteKey) || 0;
         planLineSeen.set(siteKey, occurrence + 1);
         const analysis = analyzeCallSite(index, call, name, occurrence);
@@ -808,6 +822,8 @@ function computePlanCallSites(index, name, def) {
             expression: (call.content || '').trim(),
             args: analysis.args,
             argCount: analysis.argCount,
+            ...(analysis.keywordArgNames && { keywordArgNames: analysis.keywordArgNames }),
+            ...(analysis.positionalCount != null && { positionalCount: analysis.positionalCount }),
             ...(c.calledAs && { calledAs: c.calledAs }),
         });
     }
@@ -896,7 +912,9 @@ function analyzeCallSite(index, call, funcName, occurrence = 0) {
         const targetRow = call.line - 1; // tree-sitter is 0-indexed
 
         // Find the call expression at the target line matching funcName
-        const callNode = findCallNode(tree.rootNode, callTypes, targetRow, funcName, occurrence);
+        const spelling = call.calledAs && call.calledAs !== 'bound' ? call.calledAs : funcName;
+        const callNode = findCallNode(tree.rootNode, callTypes, targetRow, spelling, occurrence,
+            call.callSite || call.provenance?.facts?.site);
         if (!callNode) return { args: null, argCount: 0 };
 
         // Check if this is a method call (obj.func()) vs a direct call (func())
@@ -1351,6 +1369,8 @@ function verify(index, name, options = {}) {
         content: c.content,
         usageType: 'call',
         receiver: c.receiver,
+        calledAs: c.calledAs,
+        callSite: c.provenance?.facts?.site,
         // Preserve receiver identity through the usage-shaped adapter. Go
         // permits a local value to have the same spelling as its type; only
         // a type-qualified call is a method expression with an explicit
@@ -1392,7 +1412,7 @@ function verify(index, name, options = {}) {
 
     const verifyLineSeen = new Map(); // 'file:line' -> per-line ordinal (fix #231)
     for (const call of calls) {
-        const siteKey = `${call.file}:${call.line}`;
+        const siteKey = `${call.file}:${call.line}:${call.calledAs || name}`;
         const occurrence = verifyLineSeen.get(siteKey) || 0;
         verifyLineSeen.set(siteKey, occurrence + 1);
         const analysis = analyzeCallSite(index, call, name, occurrence);
@@ -2158,7 +2178,10 @@ function plan(index, name, options = {}) {
             } else if (options.defaultValue) {
                 suggestion = `Add argument: ${options.defaultValue} (no default parameter values in ${planFileEntry?.language || 'this language'})`;
             } else {
-                suggestion = `Add argument: ${options.addParam}`;
+                const keywordCall = langTraits(planLang)?.keywordArguments && site.keywordArgNames?.length > 0;
+                suggestion = keywordCall
+                    ? `Add keyword argument: ${options.addParam}=${options.addParam} (replace the right-hand value with the intended expression; keep existing keyword arguments)`
+                    : `Add argument: ${options.addParam}`;
             }
             changes.push({
                 file: site.file,

package/languages/index.js

@@ -144,6 +144,7 @@ const LANGUAGES = {
         traits: {
             ...STRUCTURAL_TRAITS,
             selfParam: ['this'],
+            storedPromises: true,
             testFileCandidates: (base, ext) => [`${base}.test${ext}`, `${base}.spec${ext}`, `${base}.test.ts`, `${base}.test.js`, `${base}.spec.ts`, `${base}.spec.js`],
             testDirs: ['__tests__'],
         },
@@ -157,6 +158,7 @@ const LANGUAGES = {
         traits: {
             ...STRUCTURAL_TRAITS,
             selfParam: ['this'],
+            storedPromises: true,
             testFileCandidates: (base, ext) => [`${base}.test${ext}`, `${base}.spec${ext}`, `${base}.test.ts`, `${base}.test.js`, `${base}.spec.ts`, `${base}.spec.js`],
             testDirs: ['__tests__'],
         },
@@ -170,6 +172,7 @@ const LANGUAGES = {
         traits: {
             ...STRUCTURAL_TRAITS,
             selfParam: ['this'],
+            storedPromises: true,
             testFileCandidates: (base, ext) => [`${base}.test${ext}`, `${base}.spec${ext}`, `${base}.test.ts`, `${base}.test.js`, `${base}.spec.ts`, `${base}.spec.js`],
             testDirs: ['__tests__'],
         },
@@ -324,6 +327,7 @@ const LANGUAGES = {
         traits: {
             ...STRUCTURAL_TRAITS,
             selfParam: ['this'],
+            storedPromises: true,
             testFileCandidates: (base, ext) => [`${base}.test${ext}`, `${base}.spec${ext}`],
         },
     }

package/languages/python.js

@@ -2805,7 +2805,14 @@ function findCallsInCode(code, parser) {
             const argsNode = node.childForFieldName('arguments');
             if (argsNode) {
                 for (let i = 0; i < argsNode.namedChildCount; i++) {
-                    const arg = argsNode.namedChild(i);
+                    const rawArg = argsNode.namedChild(i);
+                    const arg = rawArg.type === 'keyword_argument'
+                        ? rawArg.childForFieldName('value') : rawArg;
+                    if (!arg) continue;
+                    // Bare keyword values already belong to the reference
+                    // inventory/rename path; only extend the member-value
+                    // callback model to match its positional counterpart.
+                    if (rawArg.type === 'keyword_argument' && arg.type !== 'attribute') continue;
                     if (arg.type === 'identifier' && !PYTHON_SKIP.has(arg.text) && !nonCallableNames.has(arg.text)) {
                         calls.push({
                             name: arg.text,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "ucn",
-  "version": "5.3.8",
+  "version": "5.4.1",
   "mcpName": "io.github.mleoca/ucn",
   "description": "Auditable AST code intelligence for AI agents: 18 task-oriented commands through one MCP tool, CLI, or agent skill. Supports JS/TS, Python, Go, Rust, Java, C, C++, C#, and HTML.",
   "main": "index.js",
@@ -10,7 +10,7 @@
   },
   "scripts": {
     "version": "node scripts/sync-server-version.js && git add server.json",
-    "test": "node --test test/audit-5.3.7.test.js test/audit-5.3.6.test.js test/evidence-provenance.test.js test/provenance-unit.test.js test/shell-output.test.js test/parser-unit.test.js test/integration.test.js test/cache.test.js test/formatter.test.js test/interactive.test.js test/feature.test.js test/regression-js.test.js test/regression-py.test.js test/regression-go.test.js test/regression-java.test.js test/regression-rust.test.js test/regression-c-family.test.js test/regression-cross.test.js test/regression-mcp.test.js test/mcp-protocol.test.js test/mcp-sdk-compat.test.js test/dependency-security.test.js test/regression-parser.test.js test/regression-commands.test.js test/regression-fixes.test.js test/regression-bugfixes.test.js test/release-surface-regressions.test.js test/prerelease-audit.test.js test/release-readiness-audit.test.js test/cross-language.test.js test/accuracy.test.js test/command-coverage.test.js test/perf-optimizations.test.js test/performance-gate-policy.test.js test/oracle-gate-policy.test.js test/outcome-policy.test.js test/consistency-eval.test.js test/agent-public-surface-benchmark.test.js test/command-contracts.test.js test/language-adapter.test.js test/systematic-test.js test/mcp-edge-cases.js test/conservation.test.js test/parity-test.js test/trust-matrix.test.js",
+    "test": "node --test test/audit-5.4.0.test.js test/audit-5.3.8.test.js test/audit-5.3.7.test.js test/audit-5.3.6.test.js test/evidence-provenance.test.js test/provenance-unit.test.js test/shell-output.test.js test/parser-unit.test.js test/integration.test.js test/cache.test.js test/formatter.test.js test/interactive.test.js test/feature.test.js test/regression-js.test.js test/regression-py.test.js test/regression-go.test.js test/regression-java.test.js test/regression-rust.test.js test/regression-c-family.test.js test/regression-cross.test.js test/regression-mcp.test.js test/mcp-protocol.test.js test/mcp-sdk-compat.test.js test/dependency-security.test.js test/regression-parser.test.js test/regression-commands.test.js test/regression-fixes.test.js test/regression-bugfixes.test.js test/release-surface-regressions.test.js test/prerelease-audit.test.js test/release-readiness-audit.test.js test/cross-language.test.js test/accuracy.test.js test/command-coverage.test.js test/perf-optimizations.test.js test/performance-gate-policy.test.js test/oracle-gate-policy.test.js test/outcome-policy.test.js test/consistency-eval.test.js test/agent-public-surface-benchmark.test.js test/command-contracts.test.js test/language-adapter.test.js test/systematic-test.js test/mcp-edge-cases.js test/conservation.test.js test/parity-test.js test/trust-matrix.test.js",
     "benchmark:agent": "node test/agent-public-surface-benchmark.js",
     "benchmark:agent:gate": "node test/agent-public-surface-benchmark.js --gate",
     "benchmark:agent:legacy": "node test/agent-understanding-benchmark.js",