Close ticket "Useless history in new session page?" (#1495): resolved by #1536, sweep stale LOGS.md comments - #1586
Merged
Conversation
… by #1536 The questioned history panel (ProjectLogPanel) and the LOGS.md writer were deleted by #1536, which resolves the ticket's gripes: no history section on the launcher, no wordy caption, no [Build] tag, and only one history surface left (the sidebar rail). The rail's STOPPED badge stays as-is — honest status wording is deliberate (a stopped run is not a done run). Sweep the comments that still described LOGS.md as existing: the launcher header, the dashboard read-model and projection notes, the delete-session leaves-behind lists, the gitignore allowlist note, the agent-store dir doc, the cli settle/prompt-agent docs, the handoff paper-trail predicate doc, the dashboard README, and format-date's examples. Deliberate past-tense B3 history notes and legacy-repo test fixtures stay. Remove the ticket, its plan, and its lock per the ticketing convention. Co-Authored-By: Claude <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01197dFdmEoxKR5eymD2aUkV
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
suleimansh
added a commit
that referenced
this pull request
Aug 18, 2026
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
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.
Closes out the ticket for #1495 per its plan: the questioned history panel (
ProjectLogPanel) and theLOGS.mdwriter were already deleted by #1536, which resolves four of the ticket's five gripes outright; the fifth (STOPPEDas noise) deliberately stays — the sidebar rail's honest status wording means "a human aborted this run", and collapsing it intoDONEwould misreport aborted runs as successful.What this PR does:
LOGS.mdcomments the plan listed — each one still describedLOGS.mdas existing and was a small lie to the next reader: the launcher header (ProjectHome.tsx), the dashboard read-model and projection notes (reads.ts,dashboard.ts,overview.ts), the delete-session leaves-behind lists (worktrees.ts,control.ts), the gitignore allowlist note (project-presets.ts), the agent-store directory doc, the twocli.tsdocs, the handoff paper-trail predicate (agent-handoff.ts), the dashboard README, andformat-date's examples.projects.ts,framework-gitignore.ts,projects.test.ts) and legacy-repo test fixtures (project-presets.test.ts,control.test.ts,agent-handoff.test.ts) are accurate as written.TODO_AGENTS.md.Issue #1495 was already closed (via the triage PR #1576); a comment documenting the resolution rationale is now on the issue.
Validation:
pnpm typecheckgreen; dashboard vitest suite green (81 files, 774 tests, includes the editedformat-date.test.ts). Nine node-test failures are unrelated to this comment-only diff: the daemon tests 503 because the dashboard bundle isn't built in this environment, andcontrol.test's tracked-runtime-state check fails because harness commita8944c8on this branch tracked.the-framework/<user>/sessions/*files — pre-existing before this change.🤖 Generated with Claude Code
https://claude.ai/code/session_01197dFdmEoxKR5eymD2aUkV
Generated by Claude Code