Track D: structure Pine revision field summaries

Author: TayDa64

docs/CHAT_CONTINUITY_IMPLEMENTATION_PLAN.md

@@ -1339,7 +1339,7 @@ The findings below are grounded in current repo seams, especially:
 
 ### Track A — Intent-safe reliability rewrites
 
-**Status:** First slice completed in working tree
+**Status:** Second slice completed in working tree
 
 **Delivered so far**
 - hardened `extractRequestedAppName(...)` in `src/main/ai-service.js` so passive open-state phrasing no longer gets treated as app-launch intent
@@ -1679,6 +1679,7 @@ This is the next Pine-facing implementation slice after the current Logs / Profi
 **Delivered so far**
 - extended `src/main/tradingview/pine-workflows.js` with a `provenance-summary` evidence mode for `pine-version-history`
 - Version History metadata requests such as `summarize the top visible revision metadata` now preserve or auto-append bounded `get_text` provenance-summary readback
+- `get_text` provenance-summary results now attach deterministic visible revision metadata such as latest visible revision label, latest visible relative time, visible revision count, and visible recency signal
 - extended prompt/seam/execution coverage in:
   - `src/main/ai-service/message-builder.js`
   - `scripts/test-tradingview-pine-data-workflows.js`

scripts/test-bug-fixes.js

@@ -251,6 +251,7 @@ test('pine workflow encodes diagnostics and compile-result evidence modes', () =
   assert(pineWorkflowContent.includes('diagnostics and warnings text'), 'Pine workflows should use diagnostics-specific readback wording');
   assert(pineWorkflowContent.includes('provenance-summary'), 'Pine workflows should support version-history provenance-summary evidence mode');
   assert(pineWorkflowContent.includes('top visible Pine Version History revision metadata'), 'Pine workflows should use provenance-summary-specific readback wording');
+  assert(pineWorkflowContent.includes('pineSummaryFields'), 'Pine workflows should carry explicit structured summary fields for provenance summaries');
 });
 
 test('system prompt includes Pine diagnostics guidance', () => {
@@ -262,6 +263,7 @@ test('system prompt includes Pine diagnostics guidance', () => {
   assert(systemPromptContent.includes('TradingView Pine diagnostics rule'), 'System prompt should include Pine diagnostics guidance');
   assert(systemPromptContent.includes('visible revision/provenance details'), 'System prompt should steer Pine provenance requests toward verified Version History text');
   assert(systemPromptContent.includes('treat visible Pine Version History entries as bounded audit/provenance evidence only'), 'Pine provenance guidance should prevent overclaiming from visible revision history');
+  assert(systemPromptContent.includes('latest visible revision label'), 'Pine provenance guidance should mention structured visible revision fields');
   assert(systemPromptContent.includes('compile success'), 'System prompt should mention compile success bounds');
   assert(systemPromptContent.includes('realtime rollback'), 'System prompt should mention Pine execution-model caveats');
 });

scripts/test-pine-diagnostics-bounds.js

@@ -70,6 +70,7 @@ async function main() {
     assert(evidenceMessage, 'pine evidence block should be injected');
     assert(evidenceMessage.content.includes('requestKind: provenance-summary'));
     assert(evidenceMessage.content.includes('Treat Pine Version History as bounded provenance evidence only'));
+    assert(evidenceMessage.content.includes('latest visible revision label, latest visible relative time, visible revision count, and visible recency signal'));
     assert(evidenceMessage.content.includes('Do not infer hidden diffs, full script history, authorship, or runtime/chart behavior from the visible revision list alone.'));
   });
 }

scripts/test-tradingview-pine-data-workflows.js

@@ -262,6 +262,13 @@ test('open pine version history and summarize visible revision metadata stays ve
   assert.strictEqual(rewritten[4].type, 'get_text');
   assert.strictEqual(rewritten[4].text, 'Pine Version History');
   assert.strictEqual(rewritten[4].pineEvidenceMode, 'provenance-summary');
+  assert.deepStrictEqual(rewritten[4].pineSummaryFields, [
+    'latest-revision-label',
+    'latest-relative-time',
+    'visible-revision-count',
+    'visible-recency-signal',
+    'top-visible-revisions'
+  ]);
   assert(/top visible Pine Version History revision metadata/i.test(rewritten[4].reason), 'version-history metadata readback should use provenance-summary wording');
 });
 
@@ -338,6 +345,13 @@ test('pine version history metadata workflow preserves trailing get_text read st
   assert.strictEqual(readSteps.length, 1, 'explicit version-history metadata readback step should be preserved without duplication');
   assert.strictEqual(readSteps[0].text, 'Pine Version History');
   assert.strictEqual(readSteps[0].pineEvidenceMode, 'provenance-summary');
+  assert.deepStrictEqual(readSteps[0].pineSummaryFields, [
+    'latest-revision-label',
+    'latest-relative-time',
+    'visible-revision-count',
+    'visible-recency-signal',
+    'top-visible-revisions'
+  ]);
   assert.strictEqual(rewritten[2].verify.target, 'pine-version-history');
 });
 

scripts/test-windows-observation-flow.js

@@ -578,6 +578,14 @@ async function run() {
       assert.deepStrictEqual(evidenceModes, ['provenance-summary'], 'Version History metadata workflow should preserve provenance-summary evidence mode');
       assert.strictEqual(execResult.observationCheckpoints[0].verified, true, 'Pine Version History panel observation should pass');
       assert.strictEqual(execResult.results[2].text, 'Revision 18 saved 2m ago; Revision 17 saved 18m ago; showing 2 visible revisions', 'Version History metadata text should be preserved on the get_text result');
+      assert.strictEqual(execResult.results[2].pineStructuredSummary.latestVisibleRevisionLabel, 'Revision 18', 'Version History metadata summary should expose the latest visible revision label');
+      assert.strictEqual(execResult.results[2].pineStructuredSummary.latestVisibleRelativeTime, '2m ago', 'Version History metadata summary should expose the latest visible relative time');
+      assert.strictEqual(execResult.results[2].pineStructuredSummary.visibleRevisionCount, 2, 'Version History metadata summary should expose the visible revision count');
+      assert.strictEqual(execResult.results[2].pineStructuredSummary.visibleRecencySignal, 'recent-churn-visible', 'Version History metadata summary should expose a bounded visible recency signal');
+      assert.deepStrictEqual(execResult.results[2].pineStructuredSummary.topVisibleRevisions, [
+        { label: 'Revision 18', relativeTime: '2m ago', revisionNumber: 18 },
+        { label: 'Revision 17', relativeTime: '18m ago', revisionNumber: 17 }
+      ], 'Version History metadata summary should expose compact top visible revisions');
       assert(!execResult.screenshotCaptured, 'Pine Version History metadata gathering should not require a screenshot loop');
     });
   });

src/main/ai-service/message-builder.js

@@ -85,6 +85,7 @@ function buildPineEvidenceConstraint({ foreground, userMessage }) {
 
   if (requestKind === 'provenance-summary') {
     lines.push('- Rule: Treat Pine Version History as bounded provenance evidence only; summarize only the top visible revision labels, relative times, and other metadata that are directly visible.');
+    lines.push('- Rule: When possible, structure the summary into compact visible fields such as latest visible revision label, latest visible relative time, visible revision count, and visible recency signal.');
     lines.push('- Rule: Do not infer hidden diffs, full script history, authorship, or runtime/chart behavior from the visible revision list alone.');
   }
 

src/main/ai-service/system-prompt.js

@@ -229,7 +229,7 @@ When the user asks you to DO something, respond with a JSON action block:
 - **Continuity rule**: if the active page title or recent action output indicates the requested browser objective is already achieved, acknowledge completion and avoid proposing additional screenshot steps.
 - **TradingView Pine evidence rule**: if the user wants concrete Pine output, errors, profiler-style evidence, visible Pine Editor status/output, or visible revision/provenance details, prefer \`open/show Pine Editor, Logs, Profiler, or Version History\` + verified panel opening + \`get_text\` before relying on screenshot analysis.
 - **TradingView Pine diagnostics rule**: treat visible Pine Editor compile results, compiler errors, warnings, and diagnostics as bounded text evidence. Do not turn \`no errors\` into claims about runtime correctness, market validity, or trading edge.
-- **TradingView Pine provenance rule**: treat visible Pine Version History entries as bounded audit/provenance evidence only. Summarize top visible revision labels, relative times, and other directly visible metadata, but do not infer hidden diffs, full script history, authorship, or runtime/chart behavior from the visible list alone.
+- **TradingView Pine provenance rule**: treat visible Pine Version History entries as bounded audit/provenance evidence only. Summarize top visible revision labels, latest visible revision label, latest visible relative time, visible revision count, visible recency signal, and other directly visible metadata, but do not infer hidden diffs, full script history, authorship, or runtime/chart behavior from the visible list alone.
 - **TradingView Pine line-budget rule**: Pine scripts are limited to 500 lines. Do not propose pasting or generating Pine scripts longer than 500 lines; prefer bounded edits, read visible line/status hints first when needed, and mention the limit explicitly when it affects read/write guidance.
 - **If you need to interact with web content inside an app** (like VS Code panels, browser tabs): Use keyboard shortcuts or coordinate-based clicks since web UI may not appear in UIA tree
 

src/main/system-automation.js

@@ -312,6 +312,105 @@ function executePowerShell(command) {
   });
 }
 
+function normalizeCompactText(value, maxLength = 240) {
+  return String(value || '').replace(/\s+/g, ' ').trim().slice(0, maxLength) || null;
+}
+
+function parseRelativeTimeToMinutes(value) {
+  const text = normalizeCompactText(value, 80);
+  if (!text) return null;
+  const match = text.match(/(\d+)\s*(s|sec|secs|second|seconds|m|min|mins|minute|minutes|h|hr|hrs|hour|hours|d|day|days|w|wk|wks|week|weeks)\s+ago/i);
+  if (!match) return null;
+
+  const amount = Number(match[1]);
+  const unit = match[2].toLowerCase();
+  if (!Number.isFinite(amount)) return null;
+
+  if (unit.startsWith('s')) return Math.max(1, amount / 60);
+  if (unit.startsWith('m')) return amount;
+  if (unit.startsWith('h')) return amount * 60;
+  if (unit.startsWith('d')) return amount * 60 * 24;
+  if (unit.startsWith('w')) return amount * 60 * 24 * 7;
+  return null;
+}
+
+function inferVisibleRevisionRecencySignal(minutes) {
+  if (!Number.isFinite(minutes)) return 'unknown-visible-recency';
+  if (minutes <= 60) return 'recent-churn-visible';
+  if (minutes <= 1440) return 'same-day-visible';
+  if (minutes >= 10080) return 'stable-visible';
+  return 'moderate-visible';
+}
+
+function buildPineVersionHistoryStructuredSummary(text, summaryFields = []) {
+  const rawText = normalizeCompactText(text, 2000);
+  if (!rawText) return null;
+
+  const revisionSegments = rawText
+    .split(/[;\n]+/)
+    .map((segment) => normalizeCompactText(segment, 280))
+    .filter(Boolean);
+
+  const visibleRevisions = revisionSegments
+    .map((segment) => {
+      const match = segment.match(/^(Revision\s+#?\s*\d+)\b(?:.*?\b(?:saved|updated|created)\s+(.+?ago))?$/i);
+      if (!match) return null;
+
+      const label = normalizeCompactText(match[1], 80);
+      const relativeTime = normalizeCompactText(match[2], 80);
+      const revisionNumberMatch = label ? label.match(/(\d+)/) : null;
+      const revisionNumber = revisionNumberMatch ? Number(revisionNumberMatch[1]) : null;
+
+      return {
+        label,
+        revisionNumber: Number.isFinite(revisionNumber) ? revisionNumber : null,
+        relativeTime,
+        recencyMinutes: parseRelativeTimeToMinutes(relativeTime)
+      };
+    })
+    .filter(Boolean)
+    .slice(0, 5);
+
+  const visibleCountMatch = rawText.match(/showing\s+(\d+)\s+visible\s+revisions?/i);
+  const visibleRevisionCount = visibleCountMatch
+    ? Number(visibleCountMatch[1])
+    : visibleRevisions.length;
+
+  const latestVisibleRevision = visibleRevisions[0] || null;
+  const compactSummary = [
+    latestVisibleRevision?.label ? `latest=${latestVisibleRevision.label}` : null,
+    latestVisibleRevision?.relativeTime ? `saved=${latestVisibleRevision.relativeTime}` : null,
+    Number.isFinite(visibleRevisionCount) ? `visible=${visibleRevisionCount}` : null,
+    latestVisibleRevision ? `signal=${inferVisibleRevisionRecencySignal(latestVisibleRevision.recencyMinutes)}` : null
+  ].filter(Boolean).join(' | ');
+
+  const fullSummary = {
+    latestVisibleRevisionLabel: latestVisibleRevision?.label || null,
+    latestVisibleRevisionNumber: Number.isFinite(latestVisibleRevision?.revisionNumber) ? latestVisibleRevision.revisionNumber : null,
+    latestVisibleRelativeTime: latestVisibleRevision?.relativeTime || null,
+    visibleRevisionCount: Number.isFinite(visibleRevisionCount) ? visibleRevisionCount : null,
+    visibleRecencySignal: latestVisibleRevision ? inferVisibleRevisionRecencySignal(latestVisibleRevision.recencyMinutes) : 'unknown-visible-recency',
+    topVisibleRevisions: visibleRevisions.map((entry) => ({
+      label: entry.label,
+      relativeTime: entry.relativeTime,
+      revisionNumber: entry.revisionNumber
+    })),
+    compactSummary: compactSummary || null
+  };
+
+  if (!Array.isArray(summaryFields) || summaryFields.length === 0) {
+    return fullSummary;
+  }
+
+  const structured = { compactSummary: fullSummary.compactSummary };
+  if (summaryFields.includes('latest-revision-label')) structured.latestVisibleRevisionLabel = fullSummary.latestVisibleRevisionLabel;
+  if (summaryFields.includes('latest-relative-time')) structured.latestVisibleRelativeTime = fullSummary.latestVisibleRelativeTime;
+  if (summaryFields.includes('visible-revision-count')) structured.visibleRevisionCount = fullSummary.visibleRevisionCount;
+  if (summaryFields.includes('visible-recency-signal')) structured.visibleRecencySignal = fullSummary.visibleRecencySignal;
+  if (summaryFields.includes('top-visible-revisions')) structured.topVisibleRevisions = fullSummary.topVisibleRevisions;
+  return structured;
+}
+
 /**
  * Focus the desktop / unfocus Electron windows before sending keyboard input
  * This is critical for SendKeys/SendInput to reach the correct target
@@ -2388,8 +2487,13 @@ async function executeAction(action) {
           action.criteria || { text: action.text, automationId: action.automationId, controlType: action.controlType }
         );
         result = { ...result, ...gtResult };
+        if (gtResult.success
+          && action?.pineEvidenceMode === 'provenance-summary'
+          && /pine version history/i.test(String(action?.text || action?.criteria?.text || ''))) {
+          result.pineStructuredSummary = buildPineVersionHistoryStructuredSummary(gtResult.text, action.pineSummaryFields);
+        }
         result.message = gtResult.success
-          ? `Got text via ${gtResult.method}: "${(gtResult.text || '').slice(0, 50)}"`
+          ? `Got text via ${gtResult.method}: "${(gtResult.text || '').slice(0, 50)}"${result.pineStructuredSummary?.compactSummary ? ` [${result.pineStructuredSummary.compactSummary}]` : ''}`
           : `Get text failed: ${gtResult.error}`;
         break;
       }

src/main/tradingview/pine-workflows.js

@@ -15,6 +15,14 @@ function mergeUnique(values = []) {
     .filter(Boolean)));
 }
 
+const PINE_VERSION_HISTORY_SUMMARY_FIELDS = Object.freeze([
+  'latest-revision-label',
+  'latest-relative-time',
+  'visible-revision-count',
+  'visible-recency-signal',
+  'top-visible-revisions'
+]);
+
 function inferPineEvidenceReadIntent(raw = '', surfaceTarget = '') {
   const normalized = normalizeTextForMatch(raw);
   if (!normalized) return false;
@@ -109,14 +117,18 @@ function buildPineReadbackStep(surfaceTarget, evidenceMode = null) {
 
   if (surfaceTarget === 'pine-version-history') {
     const mode = evidenceMode || 'generic-provenance';
-    return {
+    const step = {
       type: 'get_text',
       text: 'Pine Version History',
       reason: mode === 'provenance-summary'
-        ? 'Read top visible Pine Version History revision metadata for a bounded provenance summary'
+        ? 'Read top visible Pine Version History revision metadata for a bounded structured provenance summary'
         : 'Read visible Pine Version History entries for bounded provenance gathering',
       pineEvidenceMode: mode
     };
+    if (mode === 'provenance-summary') {
+      step.pineSummaryFields = [...PINE_VERSION_HISTORY_SUMMARY_FIELDS];
+    }
+    return step;
   }
 
   return null;
@@ -243,6 +255,14 @@ function buildTradingViewPineWorkflowActions(intent = {}, actions = []) {
   const trailing = actions.slice(intent.openerIndex + 1)
     .filter((action) => action && typeof action === 'object' && action.type !== 'screenshot');
 
+  if (intent.surfaceTarget === 'pine-version-history' && intent.pineEvidenceMode === 'provenance-summary') {
+    trailing.forEach((action) => {
+      if (action?.type === 'get_text' && !Array.isArray(action.pineSummaryFields)) {
+        action.pineSummaryFields = [...PINE_VERSION_HISTORY_SUMMARY_FIELDS];
+      }
+    });
+  }
+
   const hasExplicitReadbackStep = trailing.some((action) => action?.type === 'get_text' || action?.type === 'find_element');
 
   if (intent.wantsEvidenceReadback && !hasExplicitReadbackStep) {