Skip to content

fix(hook): resolve Claude transcript by session id - #1415

Open
rNoz wants to merge 1 commit into
backnotprop:mainfrom
rNoz:rnoz/fix-annotate-last-session-identity
Open

fix(hook): resolve Claude transcript by session id#1415
rNoz wants to merge 1 commit into
backnotprop:mainfrom
rNoz:rnoz/fix-annotate-last-session-identity

Conversation

@rNoz

@rNoz rNoz commented Aug 28, 2026

Copy link
Copy Markdown
Contributor

Summary

  • resolve Claude transcripts by the metadata session ID across project directories instead of assuming the current working-directory slug
  • fail closed when precise or relevant invalid metadata cannot safely resolve a transcript, allowing mtime heuristics only when metadata is unavailable
  • preserve /clear handling while avoiding concurrent registered sessions

Validation

  • bun test apps/hook/server/session-log.test.ts (126 passed)
  • bun test apps/hook/server --timeout 15000 (253 passed)
  • PATH="$PWD/apps/hook/node_modules/.bin:$PATH" bun run typecheck
  • git diff --check upstream/main

Closes #985.

@rNoz
rNoz marked this pull request as ready for review August 29, 2026 01:42
@rNoz

rNoz commented Aug 29, 2026

Copy link
Copy Markdown
Contributor Author

@backnotprop I hope you like it :)

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.

/plannotator-last serves another session's message when cwd diverges from the transcript's project slug

1 participant