customable/claude-mem
1
0
Fork 0
Code Issues 1 Pull requests 5 Projects Releases 163 Packages 2 Wiki Activity Actions

fix(hooks): remove unused toolName parameter from extractTargetDirectory #181

Merged
jack merged 11 commits from fix/145-remove-toolname-param into main 2026-01-24 13:24:17 +00:00
Conversation 0 Commits 11 Files changed -
jack commented 2026-01-24 10:31:34 +00:00
Owner
Copy link

Summary

  • Remove unused toolName parameter from extractTargetDirectory function
  • Update call site to not pass the parameter
  • Function only inspects input properties (file_path, path, notebook_path)

Closes #145

Test plan

  • TypeScript check passes
## Summary - Remove unused `toolName` parameter from `extractTargetDirectory` function - Update call site to not pass the parameter - Function only inspects input properties (`file_path`, `path`, `notebook_path`) Closes #145 ## Test plan - [x] TypeScript check passes
fix(hooks): remove unused toolName parameter from extractTargetDirectory
All checks were successful
CI / check (pull_request) Successful in 1s
Details
CI / build (pull_request) Successful in 43s
Details
CI / validate-plugin (pull_request) Successful in 35s
Details
bf67c9d8fe 
The function only inspects input properties (file_path, path, notebook_path)
to determine the target directory - it never used the tool name.

Closes #145

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Merge branch 'main' into fix/145-remove-toolname-param
All checks were successful
CI / build (pull_request) Successful in 1m10s
Details
bef95c3bbf
Merge branch 'main' into fix/145-remove-toolname-param
All checks were successful
CI / build (pull_request) Successful in 1m12s
Details
eaad1465be
Merge branch 'main' into fix/145-remove-toolname-param
All checks were successful
CI / build (pull_request) Successful in 1m10s
Details
e1c92368b1
Merge branch 'main' into fix/145-remove-toolname-param
All checks were successful
CI / build (pull_request) Successful in 1m10s
Details
0c26589cbc
Merge branch 'main' into fix/145-remove-toolname-param
All checks were successful
CI / build (pull_request) Successful in 1m16s
Details
0cdb012592
Merge branch 'main' into fix/145-remove-toolname-param
All checks were successful
CI / build (pull_request) Successful in 1m18s
Details
cd723f6577
Merge branch 'main' into fix/145-remove-toolname-param
All checks were successful
CI / build (pull_request) Successful in 1m16s
Details
7f901c53f1
Merge branch 'main' into fix/145-remove-toolname-param
All checks were successful
CI / build (pull_request) Successful in 1m15s
Details
25a8c87117
Merge branch 'main' into fix/145-remove-toolname-param
All checks were successful
CI / build (pull_request) Successful in 1m17s
Details
5139aa78a5
Merge branch 'main' into fix/145-remove-toolname-param
All checks were successful
CI / build (pull_request) Successful in 1m18s
Details
047db43d11
jack merged commit d4f84681fb into main 2026-01-24 13:24:17 +00:00
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
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!181
No description provided.