Skip to content

Commit 55fe73b

Browse files
committed
Update roadmap for TradingView Pine shortcut grounding
1 parent db8e8b4 commit 55fe73b

File tree

1 file changed

+29
-5
lines changed

1 file changed

+29
-5
lines changed

docs/CHAT_CONTINUITY_IMPLEMENTATION_PLAN.md

Lines changed: 29 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1724,7 +1724,7 @@ node scripts/test-bug-fixes.js
17241724

17251725
#### Track H / Slice 3 — Safe Pine authoring defaults
17261726

1727-
**Status:** First slice completed in working tree
1727+
**Status:** Completed and committed
17281728

17291729
**Delivered so far**
17301730
- generic TradingView Pine creation requests now rewrite into inspect-first Pine Editor flows instead of defaulting to `ctrl+a` + `backspace` clear-first behavior
@@ -1768,6 +1768,13 @@ node scripts/test-bug-fixes.js
17681768

17691769
#### Track H / Slice 4 — Resume-after-confirmation re-establishes prerequisites
17701770

1771+
**Status:** Completed and committed
1772+
1773+
**Delivered so far**
1774+
- `resumeAfterConfirmation(...)` now re-establishes TradingView focus and Pine editor prerequisites before destructive edit continuation
1775+
- Pine resume prerequisite shaping explicitly re-opens or re-activates Pine Editor before assuming `ctrl+a`, destructive edit keys, or typing are still safe
1776+
- focused execution regressions now prove confirmation-resume flows do not assume ephemeral editor state or selection survived the pause
1777+
17711778
**Goal**
17721779
- after confirmation pauses, re-verify TradingView focus, Pine surface visibility, and editor-active state instead of assuming ephemeral selection/focus survived.
17731780

@@ -1792,7 +1799,14 @@ node scripts/test-bug-fixes.js
17921799

17931800
### Track I — TradingView shortcuts become app-specific tool knowledge
17941801

1795-
**Status:** Planned
1802+
**Status:** Core slice completed and committed
1803+
1804+
**Delivered so far**
1805+
- added a dedicated TradingView shortcut capability/profile helper in `src/main/tradingview/shortcut-profile.js`
1806+
- stable defaults such as `/`, `Alt+A`, `Esc`, and `Ctrl+K` are now modeled as TradingView-specific capability knowledge instead of generic desktop shortcut doctrine
1807+
- drawing bindings are explicitly marked customizable / user-confirmed, and Trading Panel / DOM execution shortcuts remain context-dependent and paper-test only
1808+
- Pine Editor no longer assumes `ctrl+e` as a stable native TradingView shortcut; Pine workflows now route Pine Editor opening through a verified TradingView quick-search / command-palette path instead of hardcoding an ungrounded opener
1809+
- TradingView Pine workflows, prompt guidance, and shortcut regressions now consult and protect that app-specific shortcut profile
17961810

17971811
**Why this track exists**
17981812
- Official TradingView shortcut documentation and third-party workflow guides show an important distinction:
@@ -1805,13 +1819,13 @@ node scripts/test-bug-fixes.js
18051819
- represent TradingView shortcut knowledge as TradingView-specific capability/profile data, not as a generic keyboard rule set.
18061820

18071821
**Primary files**
1808-
- likely new: `src/main/tradingview/shortcut-profile.js`
1822+
- `src/main/tradingview/shortcut-profile.js`
18091823
- `src/main/tradingview/pine-workflows.js`
18101824
- `src/main/tradingview/indicator-workflows.js`
18111825
- `src/main/tradingview/alert-workflows.js`
18121826
- `src/main/ai-service/system-prompt.js`
18131827
- `scripts/test-bug-fixes.js`
1814-
- likely new: `scripts/test-tradingview-shortcut-profile.js`
1828+
- `scripts/test-tradingview-shortcut-profile.js`
18151829

18161830
**Implementation checklist**
18171831
- define TradingView shortcut categories in a dedicated app-specific helper:
@@ -1826,13 +1840,23 @@ node scripts/test-bug-fixes.js
18261840
- Trading/DOM shortcuts remain advisory-safe and paper-test only
18271841

18281842
**Regression additions**
1829-
- likely new `scripts/test-tradingview-shortcut-profile.js`
1843+
- `scripts/test-tradingview-shortcut-profile.js`
18301844
- `stable default shortcuts are exposed as tradingview-specific helpers`
18311845
- `drawing shortcuts are marked customizable rather than universal`
18321846
- `trading panel shortcuts are marked context-dependent and unsafe-by-default`
1847+
- `pine editor opener is routed through TradingView quick search instead of a hardcoded native shortcut`
18331848
- `scripts/test-bug-fixes.js`
18341849
- seam assertions that system prompt and TradingView workflows use TradingView-specific shortcut guidance instead of generic assumptions
18351850

1851+
**Acceptance proof**
1852+
```powershell
1853+
node scripts/test-tradingview-shortcut-profile.js
1854+
node scripts/test-tradingview-pine-workflows.js
1855+
node scripts/test-tradingview-pine-data-workflows.js
1856+
node scripts/test-windows-observation-flow.js
1857+
node scripts/test-bug-fixes.js
1858+
```
1859+
18361860
**Acceptance criteria**
18371861
- TradingView keyboard shortcut guidance is app-specific, not global desktop doctrine
18381862
- Liku can distinguish stable defaults from customizable/contextual shortcuts before proposing automation

0 commit comments

Comments
 (0)