Skip to content

Package Claude hook shared modules with the installed plugin#66

Draft
Copilot wants to merge 5 commits into
mainfrom
copilot/fix-hook-error-cannot-find-module
Draft

Package Claude hook shared modules with the installed plugin#66
Copilot wants to merge 5 commits into
mainfrom
copilot/fix-hook-error-cannot-find-module

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented May 9, 2026 •
edited
Loading

  • Inspect OpenCode entrypoint imports, package manifest, and existing packaging tests
  • Reproduce current OpenCode test state and verify dependencies needed for validation
  • Add a regression test that simulates the installed OpenCode package layout
  • Apply the smallest packaging/runtime fix so installed OpenCode code can resolve shared modules
  • Run targeted OpenCode tests, then full repo tests and final validation

Copilot AI linked an issue May 9, 2026 that may be closed by this pull request
Hook error: Cannot find module '../../shared/feedback-signals' in akm-hook.ts (v0.7.5) #65
Open
@itlackey
Fix Claude hook shared module packaging
7e7682c 
Agent-Logs-Url: https://github.com/itlackey/akm-plugins/sessions/1a2fcf9e-1e07-4844-8471-d6e2c8311fb6

Co-authored-by: itlackey <6414031+itlackey@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix hook error: Cannot find module '../../shared/feedback-signals' Package Claude hook shared modules with the installed plugin May 9, 2026
Copilot AI requested a review from itlackey May 9, 2026 08:08
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 9, 2026 •
edited
Loading

Eval diff

  • Baseline SHA: 2aafb1681325dd87afbcd2fdea90b98c99d4e643
  • Candidate SHA: b3bd4e86aeb1b6711f6efb47b8cad3af454a9466

Metric deltas

Metric Key Baseline Candidate Δ Δ% Status
curation mean_expected_coverage 0.9125 0.9125 0 0.00% • unchanged
curation mean_reciprocal_rank 0.9208 0.9208 0 0.00% • unchanged
curation zero_hit_rate 0.0500 0.0500 0 0.00% • unchanged
context_budget claude_violations 0 0 0 0.00% • unchanged
context_budget opencode_violations 0 0 0 0.00% • unchanged
context_budget claude_drop_rate 0 0 0 0.00% • unchanged
context_budget opencode_drop_rate 0 0 0 0.00% • unchanged
feedback claude_precision 1 1 0 0.00% • unchanged
feedback claude_recall 1 1 0 0.00% • unchanged
feedback claude_polarity_flips 0 0 0 0.00% • unchanged
feedback opencode_precision 1 1 0 0.00% • unchanged
feedback opencode_recall 1 1 0 0.00% • unchanged
feedback opencode_polarity_flips 0 0 0 0.00% • unchanged
memory claude_avg_body_chars 479 588 109 22.76% ✅ improved
memory claude_name_format_violations 0 0 0 0.00% • unchanged
memory claude_secret_leakages 1 1 0 0.00% • unchanged

Plugin surface

Tools added: opencode:akm_distill, opencode:akm_info, opencode:akm_init, opencode:akm_memory, opencode:akm_proposal, opencode:akm_propose, opencode:akm_reflect
Commands added: claude:akm-distill, claude:akm-memory-audit, claude:akm-memory-candidates, claude:akm-memory-promote, claude:akm-memory-reject, claude:akm-proposal, claude:akm-propose, claude:akm-reflect, claude:akm-review-proposals, claude:akm-setup
Hooks added: claude:PostCompact, claude:PostToolBatch, claude:PreToolUse, claude:SessionEnd, claude:SubagentStart, claude:TaskCompleted, claude:TaskCreated, claude:UserPromptExpansion

No regressions.

Copilot AI and others added 3 commits May 9, 2026 08:21
@itlackey
Guard OpenCode shared module packaging
31a9d25 
Agent-Logs-Url: https://github.com/itlackey/akm-plugins/sessions/ccc45c15-7d5d-43f7-8417-ec1d1a7a9c7e

Co-authored-by: itlackey <6414031+itlackey@users.noreply.github.com>
@itlackey
Harden shared redaction parsing
b71571d 
Agent-Logs-Url: https://github.com/itlackey/akm-plugins/sessions/ccc45c15-7d5d-43f7-8417-ec1d1a7a9c7e

Co-authored-by: itlackey <6414031+itlackey@users.noreply.github.com>
@itlackey
Optimize shared redaction key checks
d86a37a 
Agent-Logs-Url: https://github.com/itlackey/akm-plugins/sessions/ccc45c15-7d5d-43f7-8417-ec1d1a7a9c7e

Co-authored-by: itlackey <6414031+itlackey@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@itlackey itlackey Awaiting requested review from itlackey

Assignees

@itlackey itlackey

Copilot code review Copilot

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Hook error: Cannot find module '../../shared/feedback-signals' in akm-hook.ts (v0.7.5)

2 participants

@itlackey