Fix member-add bug + dept-aware org matching + Cassandra cleanups#212

Merged

mliu33 merged 36 commits into

mainfrom

claude/fix-member-subscription-bug-QZhjc

May 21, 2026

Commits on May 21, 2026

docs(spec): member-add improvements + DM dedup + CreatedBy/TargetUser cleanups
claude
committed
docs(plan): implementation plan for member-add improvements + cleanups
claude
committed
feat(pipelines): add GetCapacityCheckPipeline + GetAddMemberCandidatesPipeline
claude
committed
feat(model): add SectTCName, DeptID, DeptName, DeptTCName fields to User
claude
committed
feat(room-service): add (deptId, account) index for deptId-matching pipelines
claude
committed
feat(room-worker): add ListAddMemberCandidates with per-candidate flags
claude
committed
fix(chat-frontend): dedup reply navigates directly without summaries-wait
claude
committed
refactor(cassandra): drop unused target_user column from message schema
claude
committed
feat(pipelines): match deptId alongside sectId in candidate $or
claude
committed
feat(room-service): enrichment prefers dept on overlap with Go-side combine, falls back to orgID
claude
committed
fix(room-worker): create individual room_members row on org→individual member.add
claude
committed
refactor(room-worker): OrgMemberStatus carries (Name,TCName,IsDept); lookup via member.id
claude
committed
feat(room-worker): processRemoveOrg uses dept-first tiebreak with displayOrg + orgID fallback
claude
committed
refactor(room-worker): drop Room.CreatedBy, extract reconcileRoomOnDuplicateKey, use requester-sub-exists for replay equivalence
claude
committed
test: add pipeline unit tests + strengthen DM concurrent test; polish slog + pre-alloc + godoc
claude
committed
fix: 4 review findings — reconcileRoomOnDuplicateKey idempotency, spurious sys-msg on upgrade, dept-blind membership lookups, room-service/worker display parity
claude
committed
feat(room-service): ListOrgMembers matches sectId OR deptId

vjauhari-work
and
claude
committed
chore(sast): pin runtime toolchain on sast-vuln target

vjauhari-work
and
claude
committed
feat(room-service): reject phantom org IDs and accounts at request time

vjauhari-work
and
claude
committed
refactor(room-worker): address PR #171 follow-up review findings

vjauhari-work
and
claude
committed
fix: apply branch-review findings — panic→error, backfill fail-hard, phantom-validation tests

vjauhari-work
and
claude
committed
chore(make): drop COMPOSE_BAKE/COMPOSE_PARALLEL_LIMIT from `make up`

vjauhari-work
and
claude
committed
fix: resolve coderabbit findings on PR #212

vjauhari-work
and
claude
committed
docs(client-api): document users/orgs/channels on Create Room
claude
committed
refactor(room-service,room-worker): drop unused single-row CRUD store methods
claude
committed
refactor: drop shouldRotate guard and BaseKeyVersion per PR #171 review
claude
committed
docs: drop stale createdBy / createdByAccount from Room and Create Room
claude
committed
fix: address remaining CodeRabbit findings on PR #212
claude
committed
docs(client-api): widen userCount note for create-with-initial-members
claude
committed
refactor(room-service): fold FindExistingOrgIDs dedupe into one pass
claude
committed
perf(room-service): collapse FindExistingOrgIDs into one Mongo round-trip
claude
committed
revert: drop $unionWith aggregation in FindExistingOrgIDs
claude
committed
perf(pipelines): project $lookup sub-results to {_id: 1}
claude
committed
refactor(pipelines): drop GetCapacityCheckPipeline — duplicate of GetNewMembersPipeline
claude
committed
refactor(room-worker): single-pass dept-first tiebreak in processRemoveOrg
claude
committed
fix(room-worker): preserve sub when removed org has a sibling covering the same user
claude
committed