Add TradingView shortcut capability layer

Author: TayDa64

scripts/test-bug-fixes.js

@@ -202,16 +202,18 @@ test('ui-watcher exposes active window capability snapshot', () => {
 
 test('message-builder injects active app capability context', () => {
   const messageBuilderPath = path.join(__dirname, '..', 'src', 'main', 'ai-service', 'message-builder.js');
+  const capabilityPolicyPath = path.join(__dirname, '..', 'src', 'main', 'capability-policy.js');
   const fs = require('fs');
 
   const messageBuilderContent = fs.readFileSync(messageBuilderPath, 'utf8');
+  const capabilityPolicyContent = fs.readFileSync(capabilityPolicyPath, 'utf8');
 
   assert(messageBuilderContent.includes('classifyActiveAppCapability'), 'Message builder should classify active app capability');
-  assert(messageBuilderContent.includes('## Active App Capability'), 'Message builder should inject active app capability context');
+  assert(messageBuilderContent.includes('buildCapabilityPolicySystemMessage'), 'Message builder should inject active app capability context');
   assert(messageBuilderContent.includes('visual-first-low-uia'), 'Capability context should recognize low-UIA visual-first apps');
-  assert(messageBuilderContent.includes('uia-rich'), 'Capability context should recognize UIA-rich apps');
-  assert(messageBuilderContent.includes('namedInteractiveElementCount'), 'Capability context should include UIA inventory counts');
-  assert(messageBuilderContent.includes('answer-shape:'), 'Capability context should shape control-surface answers');
+  assert(capabilityPolicyContent.includes('uia-rich'), 'Capability context should recognize UIA-rich apps');
+  assert(messageBuilderContent.includes('watcherSnapshot'), 'Capability context should include watcher/UIA inventory input');
+  assert(capabilityPolicyContent.includes('answer-shape:'), 'Capability context should shape control-surface answers');
   assert(messageBuilderContent.includes('## Pine Evidence Bounds'), 'Message builder should inject a bounded Pine diagnostics evidence block when relevant');
   assert(messageBuilderContent.includes('inferPineEvidenceRequestKind'), 'Message builder should classify Pine evidence request kinds');
   assert(messageBuilderContent.includes('runtime correctness, strategy validity, profitability, or market insight'), 'Pine evidence bounds should prevent compile success from being overclaimed');
@@ -280,7 +282,7 @@ test('pine workflow encodes diagnostics and compile-result evidence modes', () =
   assert(shortcutProfileContent.includes("'create-alert'"), 'TradingView shortcut profile should define stable alert guidance');
   assert(shortcutProfileContent.includes("'drawing-tool-binding'"), 'TradingView shortcut profile should mark drawing bindings as customizable');
   assert(shortcutProfileContent.includes("'open-dom-panel'"), 'TradingView shortcut profile should classify DOM shortcuts explicitly');
-  assert(shortcutProfileContent.includes('No stable native default should be assumed for opening Pine Editor'), 'TradingView shortcut profile should stop treating Pine Editor as a stable native shortcut');
+  assert(shortcutProfileContent.includes('No dedicated official Pine Editor opener is exposed in the PDF'), 'TradingView shortcut profile should stop treating Pine Editor as a stable native shortcut');
   assert(shortcutProfileContent.includes('buildTradingViewShortcutRoute'), 'TradingView shortcut profile should expose TradingView-specific route helpers for non-native shortcuts');
   assert(shortcutProfileContent.includes("'take-snapshot'"), 'TradingView shortcut profile should include grounded reference-only snapshot guidance');
   assert(shortcutProfileContent.includes("'add-symbol-to-watchlist'"), 'TradingView shortcut profile should include grounded watchlist shortcut guidance');
@@ -600,7 +602,7 @@ test('TradingView shortcut profile and drawing bounds are wired through promptin
   assert(alertWorkflowContent.includes("require('./shortcut-profile')"), 'Alert workflow should consume TradingView shortcut profile');
   assert(pineWorkflowContent.includes("require('./shortcut-profile')"), 'Pine workflow should consume TradingView shortcut profile');
   assert(indicatorWorkflowContent.includes("buildSearchSurfaceSelectionContract"), 'Indicator workflow should consume the shared search-surface selection contract');
-  assert(shortcutProfileContent.includes("buildSearchSurfaceSelectionContract"), 'Shortcut profile should reuse the shared search-surface selection contract for Pine quick-search routes');
+  assert(shortcutProfileContent.includes("buildTradingViewShortcutSequenceRoute"), 'Shortcut profile should expose reusable shortcut sequencing for official TradingView routes');
   assert(searchSurfaceContractsContent.includes("type: 'click_element'"), 'Shared search-surface contracts should perform semantic result selection');
   assert(claimBoundsContent.includes('buildProofCarryingAnswerPrompt'), 'Claim-bounds helper should build proof-carrying answer prompts');
   assert(messageBuilderContent.includes('buildClaimBoundConstraint'), 'Message builder should inject the answer claim contract on degraded or low-trust paths');

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

@@ -255,10 +255,16 @@ test('clipboard-only pine authoring plan rewrites into guarded continuation afte
   assert(inspectStep, 'safe authoring should inspect Pine Editor state first');
   assert.strictEqual(inspectStep.continueOnPineEditorState, 'empty-or-starter');
   assert(Array.isArray(inspectStep.continueActions) && inspectStep.continueActions.length > 0, 'safe authoring inspect step should carry continuation actions');
-  assert(inspectStep.continueActions.some((action) => action?.type === 'run_command' && /set-clipboard/i.test(String(action?.command || ''))), 'continuation should preserve clipboard preparation');
-  assert(inspectStep.continueActions.some((action) => action?.type === 'key' && String(action?.key || '').toLowerCase() === 'ctrl+v'), 'continuation should paste the prepared script');
-  assert(inspectStep.continueActions.some((action) => action?.type === 'key' && String(action?.key || '').toLowerCase() === 'ctrl+enter'), 'continuation should add the script to the chart');
-  assert(inspectStep.continueActions.some((action) => action?.type === 'get_text' && action?.pineEvidenceMode === 'compile-result'), 'continuation should gather compile-result feedback after add-to-chart');
+  assert(inspectStep.continueActions.some((action) => action?.type === 'key' && String(action?.key || '').toLowerCase() === 'ctrl+i'), 'continuation should create a fresh Pine indicator through the official shortcut chord');
+  const freshInspect = inspectStep.continueActions.find((action) => action?.type === 'get_text' && action?.pineEvidenceMode === 'safe-authoring-inspect' && Array.isArray(action?.continueActions));
+  assert(freshInspect, 'continuation should verify a fresh Pine script surface after creating a new indicator');
+  assert(freshInspect.continueActions.some((action) => action?.type === 'run_command' && /set-clipboard/i.test(String(action?.command || ''))), 'fresh-script continuation should preserve clipboard preparation');
+  assert(freshInspect.continueActions.some((action) => action?.type === 'key' && String(action?.key || '').toLowerCase() === 'ctrl+v'), 'fresh-script continuation should paste the prepared script');
+  const saveInspect = freshInspect.continueActions.find((action) => action?.type === 'get_text' && action?.pineEvidenceMode === 'save-status');
+  assert(saveInspect, 'fresh-script continuation should verify visible save status before applying');
+  assert.strictEqual(saveInspect.continueOnPineLifecycleState, 'saved-state-verified');
+  assert(saveInspect.continueActions.some((action) => action?.type === 'key' && String(action?.key || '').toLowerCase() === 'ctrl+enter'), 'save-verified continuation should add the script to the chart');
+  assert(saveInspect.continueActions.some((action) => action?.type === 'get_text' && action?.pineEvidenceMode === 'compile-result'), 'save-verified continuation should gather compile-result feedback after add-to-chart');
 });
 
 test('full ai-service rewrite handles the transcript Pine prompt without browser or timeframe derailment', () => {

scripts/test-tradingview-shortcut-profile.js

@@ -32,17 +32,22 @@ test('stable default TradingView shortcuts are exposed through the profile helpe
   const indicatorSearch = getTradingViewShortcut('indicator-search');
   const createAlert = getTradingViewShortcut('create-alert');
   const quickSearch = getTradingViewShortcut('command palette');
+  const dataWindow = getTradingViewShortcut('open-data-window');
 
   assert(indicatorSearch, 'indicator-search shortcut should exist');
   assert.strictEqual(indicatorSearch.key, '/');
   assert.strictEqual(indicatorSearch.category, 'stable-default');
+  assert.deepStrictEqual(indicatorSearch.keySequence, ['/']);
+  assert.strictEqual(indicatorSearch.automationRoutable, true);
   assert(createAlert, 'create-alert shortcut should exist');
   assert.strictEqual(createAlert.key, 'alt+a');
   assert.strictEqual(createAlert.category, 'stable-default');
   assert.strictEqual(getTradingViewShortcutKey('symbol-search'), 'ctrl+k');
   assert(quickSearch, 'symbol-search alias should resolve through the profile helper');
   assert.strictEqual(quickSearch.id, 'symbol-search');
   assert.strictEqual(quickSearch.surface, 'quick-search');
+  assert(dataWindow, 'data window shortcut should exist');
+  assert.strictEqual(dataWindow.key, 'alt+d');
 });
 
 test('drawing shortcuts are marked customizable rather than universal', () => {
@@ -81,18 +86,18 @@ test('buildTradingViewShortcutAction preserves shortcut metadata for workflow ac
 test('listTradingViewShortcuts returns the categorized TradingView profile inventory', () => {
   const shortcuts = listTradingViewShortcuts();
   assert(Array.isArray(shortcuts), 'shortcut inventory should be an array');
-  assert(shortcuts.length >= 12, 'shortcut inventory should include the grounded TradingView shortcut inventory');
+  assert(shortcuts.length >= 20, 'shortcut inventory should include the expanded TradingView shortcut inventory');
 });
 
-test('shortcut profile exposes reference-only chart shortcuts with source provenance', () => {
+test('shortcut profile exposes official chart shortcuts with source provenance', () => {
   const snapshot = getTradingViewShortcut('take snapshot');
   const watchlist = getTradingViewShortcut('add-symbol-to-watchlist');
 
   assert(snapshot, 'snapshot shortcut should resolve by alias');
   assert.strictEqual(snapshot.key, 'alt+s');
   assert.strictEqual(snapshot.category, 'reference-only');
-  assert.strictEqual(snapshot.sourceConfidence, 'secondary-reference');
-  assert(snapshot.sourceUrls.includes(TRADINGVIEW_SHORTCUTS_SECONDARY_URL));
+  assert.strictEqual(snapshot.sourceConfidence, 'official-pdf');
+  assert(snapshot.sourceUrls.includes(TRADINGVIEW_SHORTCUTS_OFFICIAL_URL));
   assert(watchlist, 'watchlist shortcut should exist');
   assert.strictEqual(watchlist.key, 'alt+w');
   assert.strictEqual(watchlist.surface, 'watchlist');
@@ -105,7 +110,7 @@ test('shortcut profile resolves aliases and documents official shortcut referenc
 
   const indicatorSearch = getTradingViewShortcut('indicator-search');
   assert(indicatorSearch.sourceUrls.includes(TRADINGVIEW_SHORTCUTS_OFFICIAL_URL));
-  assert(indicatorSearch.sourceUrls.includes(TRADINGVIEW_SHORTCUTS_SECONDARY_URL));
+  assert.strictEqual(indicatorSearch.sourceConfidence, 'official-pdf');
 });
 
 test('shortcut profile exposes reusable phrase matching helpers for workflow inference', () => {
@@ -138,3 +143,29 @@ test('pine editor opener is routed through TradingView quick search instead of a
   assert.strictEqual(routeActions[4].type, 'key');
   assert.strictEqual(routeActions[4].key, 'enter');
 });
+
+test('pine authoring shortcuts expose normalized capability metadata and chorded sequences', () => {
+  const newIndicator = getTradingViewShortcut('new-pine-indicator');
+  const saveScript = getTradingViewShortcut('save-pine-script');
+  const addToChart = getTradingViewShortcut('add-pine-to-chart');
+
+  assert(newIndicator, 'new pine indicator shortcut should exist');
+  assert.deepStrictEqual(newIndicator.keySequence, ['ctrl+k', 'ctrl+i']);
+  assert.strictEqual(newIndicator.key, null);
+  assert.strictEqual(newIndicator.automationRoutable, true);
+  assert.strictEqual(newIndicator.fallbackPolicy, 'none');
+  assert.strictEqual(saveScript.key, 'ctrl+s');
+  assert.strictEqual(saveScript.verificationContract.kind, 'status-visible');
+  assert.strictEqual(addToChart.key, 'ctrl+enter');
+  assert.strictEqual(addToChart.automationRoutable, true);
+});
+
+test('generic shortcut route builder emits a chord sequence with final verification metadata', () => {
+  const routeActions = buildTradingViewShortcutRoute('new-pine-indicator');
+  const keyActions = routeActions.filter((action) => action?.type === 'key');
+
+  assert(Array.isArray(routeActions) && routeActions.length >= 4, 'new indicator route should emit multiple steps');
+  assert.deepStrictEqual(keyActions.map((action) => action.key), ['ctrl+k', 'ctrl+i']);
+  assert.strictEqual(keyActions[1].verify.kind, 'editor-active');
+  assert.strictEqual(keyActions[1].tradingViewShortcut.id, 'new-pine-indicator');
+});