cleanup(viewer): drop "plans" from the memory/work-files names - #136
Merged
Conversation
JeanBaptisteRenard
enabled auto-merge (squash)
August 21, 2026 14:31
#138 already removed the Plans tab, its viewer panel, the get-plans/read-plan/save-plan IPC handlers and the 'plan' FTS type. What it left behind is the naming: public/plans-memory-view.js and docs/plans-memory-workfiles.md still carried "plans" in their names, and several comments and doc tables still listed a panel that no longer exists. Rename both files to memory-workfiles-view.js / memory-workfiles.md and update every reference. Restore the cross-tab search test that #138 deleted along with its plans case, retargeted at the memory tab: it covers the same sub-3-char routing branch, which was left untested. Left untouched: the shared .plans-empty CSS class, reused by Memory, Work Files, Stats and jsonl-viewer; the 'plan' permission mode; and the 'plan' subagent type colour.
devsuitup
force-pushed
the
cleanup/remove-plans-panel
branch
from
August 23, 2026 20:29
ee1da1c to
8a85928
Compare
devsuitup
changed the title
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The Plans tab itself was removed in #138. What was left over is the naming: the renderer file and the doc page were still called
plans-*although they no longer serve any plan.public/plans-memory-view.js→public/memory-workfiles-view.js(content unchanged apart from the globals header)docs/plans-memory-workfiles.md→docs/memory-workfiles.md(content identical)index.html,app.js,jsonl-viewer.js,README.md,docs/README.md,.ai/contexts/*,eslint.config.jsand the two DOM testsstyle.css,viewer-panel.js,viewer-toolbar.jsAlso restores one test #138 dropped: the cross-tab routing assertion below the 3-character search threshold only existed in its "plans" variant, so removing the tab removed the coverage with it. Re-added against the memory tab — same code path.
No behaviour change.
npm test: 672 tests, 665 pass, 0 fail, 7 pre-existing skips.eslint: 0 errors.The
.plans-emptyCSS class is deliberately left alone: it is shared by the Stats, JSONL and Memory empty states, and renaming it is a separate change.