refactor(hooks): remove unused logger and isReady from client.ts #179

Merged

jack merged 13 commits from refactor/148-cleanup-hooks-client into main 2026-01-24 13:28:07 +00:00

Conversation 0 Commits 13 Files changed -

jack commented 2026-01-24 10:30:15 +00:00

Owner

Copy link

Summary

• Remove unused logger import and constant (never called)
• Remove unused isReady() method (hooks only use isCoreReady())

Closes #148

Test plan

• TypeScript check passes
• Verified logger/isReady not used via grep

## Summary - Remove unused `logger` import and constant (never called) - Remove unused `isReady()` method (hooks only use `isCoreReady()`) Closes #148 ## Test plan - [x] TypeScript check passes - [x] Verified logger/isReady not used via grep

jack added 1 commit 2026-01-24 10:30:15 +00:00

refactor(hooks): remove unused logger and isReady from client.ts ... fe3c4f1bc2

- Remove unused logger import and constant
- Remove isReady() method (hooks only use isCoreReady())

Closes #148

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

jack added the

priority

low

type

refactor

labels 2026-01-24 10:30:17 +00:00

jonas.hanisch added 1 commit 2026-01-24 12:45:05 +00:00

Merge branch 'main' into refactor/148-cleanup-hooks-client d6c0e5d7f3

jack added 1 commit 2026-01-24 12:53:03 +00:00

Merge branch 'main' into refactor/148-cleanup-hooks-client a2d26e3ec7

jack added 1 commit 2026-01-24 12:56:07 +00:00

Merge branch 'main' into refactor/148-cleanup-hooks-client 5b6d457b59

jack added 1 commit 2026-01-24 13:00:09 +00:00

Merge branch 'main' into refactor/148-cleanup-hooks-client cd27e43bc6

jack added 1 commit 2026-01-24 13:04:06 +00:00

Merge branch 'main' into refactor/148-cleanup-hooks-client c48067671a

jack added 1 commit 2026-01-24 13:06:27 +00:00

Merge branch 'main' into refactor/148-cleanup-hooks-client 8e24814efe

jack added 1 commit 2026-01-24 13:08:06 +00:00

Merge branch 'main' into refactor/148-cleanup-hooks-client 3f50032fbe

jack added 1 commit 2026-01-24 13:14:12 +00:00

Merge branch 'main' into refactor/148-cleanup-hooks-client c116e00e81

jack added 1 commit 2026-01-24 13:18:06 +00:00

Merge branch 'main' into refactor/148-cleanup-hooks-client e2546dc575

jack added 1 commit 2026-01-24 13:22:08 +00:00

Merge branch 'main' into refactor/148-cleanup-hooks-client cd7fa4fb73

jack added 1 commit 2026-01-24 13:24:18 +00:00

Merge branch 'main' into refactor/148-cleanup-hooks-client 9a4f06056e

jack added 1 commit 2026-01-24 13:26:03 +00:00

Merge branch 'main' into refactor/148-cleanup-hooks-client a3c38e5529

jack merged commit 0666f01e5e into main 2026-01-24 13:28:07 +00:00

jack referenced this pull request from a commit 2026-01-24 13:28:08 +00:00

Merge pull request 'refactor(hooks): remove unused logger and isReady from client.ts' (#179) from refactor/148-cleanup-hooks-client into main

Sign in to join this conversation.

Reviewers

No reviewers

Labels

Clear labels   

good first issue

Good for newcomers

  

has-pr

Issue has an associated pull request

  

help wanted

Community contributions welcome

  

idea

Idea or concept - not yet ready for implementation

  

priority

critical

Urgent issue requiring immediate attention

  

priority

high

High priority issue

  

priority

low

Low priority issue

  

priority

medium

Medium priority issue

  

status

blocked

Blocked by external dependency

  

status

in-progress

Currently being worked on

  

status

needs-review

Awaiting code review

  

status

ready

Ready to be worked on

  

type

bug

Something isn't working

  

type

docs

Documentation improvements

  

type

enhancement

Improvement to existing functionality

  

type

feature

New feature or request

  

type

refactor

Code refactoring without functional changes

No labels

good first issue

has-pr

help wanted

idea

priority

critical

priority

high

priority

low

priority

medium

status

blocked

status

in-progress

status

needs-review

status

ready

type

bug

type

docs

type

enhancement

type

feature

type

refactor

Milestone

Clear milestone

No items

No milestone

Projects

Clear projects

No items

No project

Assignees

Clear assignees

No assignees

2 participants

Notifications

Subscribe

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference

customable/claude-mem!179

No description provided.