Skip to content

fix(zsxq): require active group context#579

Merged
jackwener merged 2 commits intomainfrom
codex/zsxq-active-group
Mar 29, 2026
Merged

fix(zsxq): require active group context#579
jackwener merged 2 commits intomainfrom
codex/zsxq-active-group

Conversation

@jackwener
Copy link
Copy Markdown
Owner

Summary

  • require an active ZSXQ group context instead of silently falling back to the first joined group
  • rename the helper to getActiveGroupId to match the new behavior
  • add adapter tests for topics/search when no active group context exists

Verification

  • npx vitest run --project adapter src/clis/zsxq/topic.test.ts src/clis/zsxq/topics.test.ts src/clis/zsxq/search.test.ts
  • npm run typecheck

@jackwener jackwener merged commit f8e9b08 into main Mar 29, 2026
11 checks passed
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