@windsland52/maa-log-tools 1.2.0 → 1.2.2

package/README.md

@@ -71,7 +71,7 @@ process-start-bounded interval contains the timestamp; otherwise it returns `nul
 
 ## Runtime inspection
 
-`buildRuntimeInspection(kernelOutput, frameworkExtraction, sourceSegments?)` emits
+`buildRuntimeInspection(kernelOutput, frameworkExtraction, sourceSegments)` emits
 `mla-runtime-inspection/v1`. It nests task executions under their runtime session and keeps three
 different semantics separate:
 
@@ -92,9 +92,8 @@ ambiguous tasks remain in `unscopedTasks` and produce a warning.
 Use `--runtime-inspection` to emit this result directly from a file, zip, or directory. It is a
 separate output mode from `--preflight`; the two flags are mutually exclusive.
 
-When `sourceSegments` is provided, each evidence position is enriched with `source`, `path`, and
-`localLine` describing which original log file the evidence came from and its 1-based line within
-that file. Segments are built by the Node extraction helpers (`loadNodeLogDirectory`,
-`extractZipContentFromNodeFile`) when merging multiple log files; `parserInputLine` remains the
-line in the merged parser input, while `localLine` is the offset within the individual source
-file. Callers without segments leave these fields as `null`.
+Each evidence position carries `source`, `path`, and `localLine` describing which original log
+file the evidence came from and its 1-based line within that file. Segments are built by the Node
+extraction helpers (`loadNodeLogDirectory`, `extractZipContentFromNodeFile`) when merging multiple
+log files and are required by `buildRuntimeInspection`; `localLine` is the offset within the
+individual source file, not the merged parser input line.

package/dist/cli.js

@@ -108,7 +108,7 @@ export const main = async () => {
         ? extractFrameworkSessions(await loadFrameworkLogSources(resolvedPath))
         : EMPTY_FRAMEWORK_EXTRACTION;
     let result = null;
-    let sourceSegments;
+    let sourceSegments = [];
     if (targetStat.isDirectory()) {
         const extracted = await loadNodeLogDirectory(resolvedPath);
         if (extracted) {

package/dist/runtimeInspection.d.ts

@@ -3,7 +3,6 @@ import type { FrameworkSession, FrameworkSessionExtraction } from './frameworkVe
 export declare const MLA_RUNTIME_INSPECTION_SCHEMA_VERSION = "mla-runtime-inspection/v1";
 export interface RuntimeEvidencePosition {
     timestamp: string | null;
-    parserInputLine: number | null;
     source: string | null;
     path: string | null;
     localLine: number | null;
@@ -62,6 +61,15 @@ export interface RecognitionOccurrenceSample {
         end: RuntimeEvidencePosition;
     };
 }
+export interface RepeatedNodeOccurrenceSample {
+    pattern: string[];
+    firstSeenAt: string;
+    lastSeenAt: string;
+    repeatCount: number;
+    durationMs: number;
+    termination: 'left_pattern' | 'task_ended' | 'still_repeating_at_log_end';
+    evidence: RuntimeEvidencePosition;
+}
 export interface RecognitionActivitySignal extends RuntimeScope {
     signalId: string;
     kind: 'recognition_activity';
@@ -117,24 +125,9 @@ export interface RepeatedNodeSequenceSignal extends RuntimeScope {
         stillRepeatingAtLogEnd: number;
     };
     representatives: {
-        first: {
-            firstSeenAt: string;
-            lastSeenAt: string;
-            repeatCount: number;
-            evidence: RuntimeEvidencePosition;
-        };
-        longest: {
-            firstSeenAt: string;
-            lastSeenAt: string;
-            repeatCount: number;
-            evidence: RuntimeEvidencePosition;
-        };
-        last: {
-            firstSeenAt: string;
-            lastSeenAt: string;
-            repeatCount: number;
-            evidence: RuntimeEvidencePosition;
-        };
+        first: RepeatedNodeOccurrenceSample;
+        longest: RepeatedNodeOccurrenceSample;
+        last: RepeatedNodeOccurrenceSample;
     };
     detector: {
         name: 'repeated-completed-node-sequence';
@@ -221,5 +214,5 @@ export interface RuntimeInspection {
     signals: RuntimeSignal[];
     warnings: string[];
 }
-export declare const buildRuntimeInspection: (output: KernelOutput, framework: FrameworkSessionExtraction, sourceSegments?: readonly SourceSegment[]) => RuntimeInspection;
+export declare const buildRuntimeInspection: (output: KernelOutput, framework: FrameworkSessionExtraction, sourceSegments: readonly SourceSegment[]) => RuntimeInspection;
 export {};

package/dist/runtimeInspection.js

@@ -30,11 +30,11 @@ const evidence = (index, timestamp) => {
     if (timestamp) {
         const exactLine = index.exactLines.get(timestamp);
         if (exactLine != null)
-            return { timestamp, parserInputLine: exactLine, source: null, path: null, localLine: null };
+            return { timestamp, mergedLine: exactLine };
     }
     const target = timestampMs(timestamp);
     if (target == null || index.ordered.length === 0) {
-        return { timestamp: timestamp ?? null, parserInputLine: null, source: null, path: null, localLine: null };
+        return { timestamp: timestamp ?? null, mergedLine: null };
     }
     let low = 0;
     let high = index.ordered.length;
@@ -50,7 +50,7 @@ const evidence = (index, timestamp) => {
     const nearest = before == null
         ? after
         : after == null || target - before.timestamp <= after.timestamp - target ? before : after;
-    return { timestamp: timestamp ?? null, parserInputLine: nearest?.line ?? null, source: null, path: null, localLine: null };
+    return { timestamp: timestamp ?? null, mergedLine: nearest?.line ?? null };
 };
 const recognitionItems = (items) => ((items ?? []).flatMap(item => [
     ...(item.type === 'recognition' || item.type === 'recognition_node' ? [item] : []),
@@ -161,25 +161,27 @@ const canonicalCycle = (pattern) => {
     rotations.sort((left, right) => JSON.stringify(left).localeCompare(JSON.stringify(right)));
     return rotations[0] ?? [...pattern];
 };
-const findSourceSegment = (segments, parserInputLine) => {
+const findSourceSegment = (segments, mergedLine) => {
     for (const segment of segments) {
-        if (parserInputLine >= segment.startLine && parserInputLine < segment.startLine + segment.lineCount) {
+        if (mergedLine >= segment.startLine && mergedLine < segment.startLine + segment.lineCount) {
             return segment;
         }
     }
     return null;
 };
 const enrichWithSource = (position, segments) => {
-    if (!segments || position.parserInputLine == null)
-        return position;
-    const segment = findSourceSegment(segments, position.parserInputLine);
-    if (!segment)
-        return position;
+    if (position.mergedLine == null) {
+        return { timestamp: position.timestamp, source: null, path: null, localLine: null };
+    }
+    const segment = findSourceSegment(segments, position.mergedLine);
+    if (!segment) {
+        return { timestamp: position.timestamp, source: null, path: null, localLine: null };
+    }
     return {
-        ...position,
+        timestamp: position.timestamp,
         source: segment.source,
         path: segment.path,
-        localLine: position.parserInputLine - segment.startLine + 1,
+        localLine: position.mergedLine - segment.startLine + 1,
     };
 };
 const createEvidenceFn = (segments) => (index, timestamp) => enrichWithSource(evidence(index, timestamp), segments);
@@ -393,21 +395,30 @@ export const buildRuntimeInspection = (output, framework, sourceSegments) => {
             const last = completed[lastIndex];
             if (!first || !last)
                 continue;
-            const reachesEnd = lastIndex === completed.length - 1;
             const rawPattern = completed.slice(repeated.start, repeated.start + repeated.length).map(node => node.name);
             const pattern = repeated.length === 1 ? rawPattern : canonicalCycle(rawPattern);
             const kind = repeated.length === 1 ? 'repeated_node' : 'repeated_node_cycle';
             const key = JSON.stringify([kind, pattern]);
             const group = repetitionGroups.get(key) ?? [];
+            const timelineLastIndex = timeline.findIndex(item => item.nodeInfo === last);
+            const trailing = timelineLastIndex < 0 ? [] : timeline.slice(timelineLastIndex + 1);
+            const reachesCompletedEnd = lastIndex === completed.length - 1;
+            const continuesAtLogEnd = reachesCompletedEnd
+                && task.status === 'running'
+                && trailing.every((item, offset) => (item.nodeInfo.status === 'running'
+                    && item.nodeInfo.name === rawPattern[offset % rawPattern.length]));
+            const taskEndedAtPattern = reachesCompletedEnd
+                && timelineLastIndex === timeline.length - 1
+                && task.status !== 'running';
             group.push({
                 pattern,
                 repeatCount: repeated.count,
                 durationMs: elapsed(first.ts, last.end_ts ?? last.ts) ?? 0,
                 firstSeenAt: first.ts,
                 lastSeenAt: last.end_ts ?? last.ts,
-                termination: reachesEnd
-                    ? task.status === 'running' ? 'still_repeating_at_log_end' : 'task_ended'
-                    : 'left_pattern',
+                termination: continuesAtLogEnd
+                    ? 'still_repeating_at_log_end'
+                    : taskEndedAtPattern ? 'task_ended' : 'left_pattern',
                 evidence: evidenceAt(evidenceIndex, first.ts),
             });
             repetitionGroups.set(key, group);
@@ -427,8 +438,6 @@ export const buildRuntimeInspection = (output, framework, sourceSegments) => {
             const totalRepeatCount = group.reduce((sum, item) => sum + item.repeatCount, 0);
             const maximumRepeatCount = Math.max(...group.map(item => item.repeatCount));
             const repetitionReasons = [];
-            if (terminations.leftPattern > 0)
-                repetitionReasons.push('incomplete_repetition');
             if (terminations.stillRepeatingAtLogEnd > 0)
                 repetitionReasons.push('still_repeating_at_log_end');
             if (maximumRepeatCount >= 10 || totalRepeatCount >= 20)
@@ -481,8 +490,9 @@ export const buildRuntimeInspection = (output, framework, sourceSegments) => {
         const visionImages = imageSets.flatMap(set => set.vision);
         const ownFailures = failures.filter(failure => directFailureIds.includes(failure.failureId));
         const ownSignals = signals.filter(signal => signalIds.includes(signal.signalId));
-        const recognitionActivity = ownSignals
-            .filter((signal) => (signal.kind === 'recognition_activity'))
+        const recognitionSignals = ownSignals
+            .filter((signal) => (signal.kind === 'recognition_activity'));
+        const recognitionActivity = [...recognitionSignals]
             .sort((left, right) => (right.unsuccessfulAttempts.maximum - left.unsuccessfulAttempts.maximum
             || right.occurrenceCount - left.occurrenceCount))
             .slice(0, 5)
@@ -517,7 +527,7 @@ export const buildRuntimeInspection = (output, framework, sourceSegments) => {
                     const statuses = new Set(attempts.map(attempt => attempt.status));
                     return statuses.has('failed') && statuses.has('success');
                 }).length,
-                recognitionActivityGroups: recognitionActivity.length,
+                recognitionActivityGroups: recognitionSignals.length,
                 maximumRecognitionAttemptsPerNode: Math.max(0, ...attemptsByNode.map(attempts => attempts.length)),
                 maximumUnsuccessfulRecognitionAttemptsPerNode: Math.max(0, ...attemptsByNode.map(attempts => attempts.filter(attempt => attempt.status === 'failed').length)),
                 actionAttempts: timeline.filter(item => item.nodeInfo.action_details != null).length,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@windsland52/maa-log-tools",
-  "version": "1.2.0",
+  "version": "1.2.2",
   "type": "module",
   "private": false,
   "bin": {
@@ -43,9 +43,9 @@
   "dependencies": {
     "fflate": "^0.8.2",
     "@windsland52/maa-log-adapter": "1.0.1",
-    "@windsland52/maa-log-parser": "1.0.1",
     "@windsland52/maa-log-runtime": "1.0.1",
-    "@windsland52/maa-log-kernel": "1.0.1"
+    "@windsland52/maa-log-kernel": "1.0.1",
+    "@windsland52/maa-log-parser": "1.0.1"
   },
   "engines": {
     "node": ">=24.0.0"