Thanks to visit codestin.com
Credit goes to github.com

Skip to content

fix: remove debug console.log statements from BranchCard#97

Merged
matt2e merged 1 commit intomainfrom
matt2e/bad-logs
Feb 13, 2026
Merged

fix: remove debug console.log statements from BranchCard#97
matt2e merged 1 commit intomainfrom
matt2e/bad-logs

Conversation

@matt2e
Copy link
Contributor

@matt2e matt2e commented Feb 13, 2026

Remove verbose [BranchCard] debug logging for listener registration, action_status event processing, and branch filtering. Retains console.error calls for legitimate error handling.

Remove verbose [BranchCard] debug logging for listener registration,
action_status event processing, and branch filtering. Retains
console.error calls for legitimate error handling.
@matt2e matt2e merged commit 43f3fc7 into main Feb 13, 2026
3 checks passed
@matt2e matt2e deleted the matt2e/bad-logs branch February 13, 2026 17:29
taylorkmho added a commit that referenced this pull request Feb 13, 2026
* origin/main:
  fix: surface worktree setup errors instead of showing infinite spinner (#106)
  feat: store data in platform-conventional directories (XDG) (#104)
  feat(notes): add copy-to-clipboard button in NoteModal (#103)
  feat: add use github repo based projects to avoid cloning when using remote branches (#102)
  fix: always branch new worktrees from remote-tracking refs (#100)
  fix: remove debug console.log statements from BranchCard (#97)
  feat: improve PR button UX with push support and conditional visibility (#89)
loganj added a commit that referenced this pull request Feb 26, 2026
* fix: consistent file display across all pages

Show workspace-rooted path instead of bare filename on RecentPage,
SearchPage, and InReviewPage. ProjectPage keeps using base filename
since files are already shown within their source context.

- RecentPage: reuse shared file-row layout classes, remove redundant
  recent-file-specific CSS, show path as primary with project as subtitle
- SearchPage: show path instead of name as primary display
- InReviewPage: add title fallback and subtitle span
- ProjectPage in-review section: add title fallback and subtitle span
- Backend: add Title field to ReviewFileEntry, populate from cache

Co-Authored-By: Claude Opus 4.6 <[email protected]>

* test: add title/subtitle presentation tests for all file-rendering pages

Each page now has a test verifying that when a file has a title set,
the title is shown as primary text and the path (or name on ProjectPage)
is shown as the subtitle.

Co-Authored-By: Claude Opus 4.6 <[email protected]>

* fix: show project in subtitle for titled files on RecentPage

When a file has a title, the subtitle now shows project/path instead of
just path, so files from different projects remain distinguishable in the
cross-project recent files list.

Co-Authored-By: Claude Opus 4.6 <[email protected]>

---------

Co-authored-by: Claude Opus 4.6 <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant