This repository was archived by the owner on Feb 25, 2026. It is now read-only.
Commit 1ca23dd
committed
fix(vscode): display sub-agent task tool list in webview
The task tool's child session data was never populated in the VSCode
webview's DataProvider store, causing the task-tools list to render
empty. Two root causes:
1. DataBridge only mapped the current session's messages/parts, so
getSessionToolParts() always returned [] for child session IDs.
2. No syncSession callback was provided to DataProvider, so the task
tool renderer's createEffect that triggers child session loading
was a no-op.
Add a syncSession message round-trip: the webview sends syncSession to
the extension, which tracks the child session ID for SSE events and
fetches its messages via HTTP, sending them back as messagesLoaded.
The DataBridge now exposes the full allMessages/allParts store so all
sessions (including child sessions) are visible to the shared
message-part renderer.1 parent 4b7e20b commit 1ca23dd
4 files changed
Lines changed: 75 additions & 11 deletions
File tree
- packages/kilo-vscode
- src
- webview-ui/src
- context
- types
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
263 | 263 | | |
264 | 264 | | |
265 | 265 | | |
| 266 | + | |
| 267 | + | |
| 268 | + | |
266 | 269 | | |
267 | 270 | | |
268 | 271 | | |
| |||
579 | 582 | | |
580 | 583 | | |
581 | 584 | | |
| 585 | + | |
| 586 | + | |
| 587 | + | |
| 588 | + | |
| 589 | + | |
| 590 | + | |
| 591 | + | |
| 592 | + | |
| 593 | + | |
| 594 | + | |
| 595 | + | |
| 596 | + | |
| 597 | + | |
| 598 | + | |
| 599 | + | |
| 600 | + | |
| 601 | + | |
| 602 | + | |
| 603 | + | |
| 604 | + | |
| 605 | + | |
| 606 | + | |
| 607 | + | |
| 608 | + | |
| 609 | + | |
| 610 | + | |
| 611 | + | |
| 612 | + | |
| 613 | + | |
| 614 | + | |
| 615 | + | |
| 616 | + | |
| 617 | + | |
| 618 | + | |
| 619 | + | |
| 620 | + | |
| 621 | + | |
| 622 | + | |
582 | 623 | | |
583 | 624 | | |
584 | 625 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
50 | 50 | | |
51 | 51 | | |
52 | 52 | | |
| 53 | + | |
| 54 | + | |
53 | 55 | | |
54 | | - | |
| 56 | + | |
55 | 57 | | |
56 | 58 | | |
57 | | - | |
58 | | - | |
59 | | - | |
60 | | - | |
61 | | - | |
62 | | - | |
63 | | - | |
64 | | - | |
65 | | - | |
| 59 | + | |
| 60 | + | |
66 | 61 | | |
67 | 62 | | |
68 | 63 | | |
| |||
71 | 66 | | |
72 | 67 | | |
73 | 68 | | |
| 69 | + | |
| 70 | + | |
| 71 | + | |
| 72 | + | |
74 | 73 | | |
75 | | - | |
| 74 | + | |
76 | 75 | | |
77 | 76 | | |
78 | 77 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
102 | 102 | | |
103 | 103 | | |
104 | 104 | | |
| 105 | + | |
| 106 | + | |
| 107 | + | |
| 108 | + | |
| 109 | + | |
| 110 | + | |
105 | 111 | | |
106 | 112 | | |
107 | 113 | | |
| |||
143 | 149 | | |
144 | 150 | | |
145 | 151 | | |
| 152 | + | |
146 | 153 | | |
147 | 154 | | |
148 | 155 | | |
| |||
790 | 797 | | |
791 | 798 | | |
792 | 799 | | |
| 800 | + | |
| 801 | + | |
| 802 | + | |
| 803 | + | |
| 804 | + | |
| 805 | + | |
| 806 | + | |
| 807 | + | |
793 | 808 | | |
794 | 809 | | |
795 | 810 | | |
| |||
865 | 880 | | |
866 | 881 | | |
867 | 882 | | |
| 883 | + | |
| 884 | + | |
868 | 885 | | |
869 | 886 | | |
870 | 887 | | |
| |||
877 | 894 | | |
878 | 895 | | |
879 | 896 | | |
| 897 | + | |
880 | 898 | | |
881 | 899 | | |
882 | 900 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
729 | 729 | | |
730 | 730 | | |
731 | 731 | | |
| 732 | + | |
| 733 | + | |
| 734 | + | |
| 735 | + | |
| 736 | + | |
732 | 737 | | |
733 | 738 | | |
734 | 739 | | |
| |||
772 | 777 | | |
773 | 778 | | |
774 | 779 | | |
| 780 | + | |
775 | 781 | | |
776 | 782 | | |
777 | 783 | | |
| |||
0 commit comments