mirror of
https://github.com/CopilotKit/CopilotKit.git
synced 2026-09-14 16:26:20 +08:00
10587a8cd6
## What Three comments in the channels packages point at Linear tickets that were **canceled on 2026-07-01** (superseded by OSS-401's consolidated realtime foundation). They read as live tracking pointers when nothing actually tracks the work. | File | Reference | Ticket status | |---|---|---| | `packages/channels-core/src/codec.ts` | `TODO(OSS-363)` | Canceled | | `packages/channels-intelligence/src/runtime.ts` | `TODO(OSS-377)` | Canceled | | `packages/channels-slack/src/ingress-normalize.ts` | `(OSS-362)` provenance note | Canceled | ## How The engineering intent is **preserved** in each case and re-marked as `TODO (untracked)` with a note that the original ticket was canceled — so a reader re-files rather than chasing a dead ticket. The `OSS-362` mention was a bare provenance parenthetical carrying no intent, so it is simply dropped. `codec.ts` additionally records that the TODO's stated precondition is **already met**: the pure Slack ingress mapping (mention stripping, stable event-id derivation, real-user filtering, field extraction) now lives in `channels-slack/src/ingress-normalize.ts`. The remaining work is hoisting `normalizeIngress` onto the generic `PlatformCodec` — worth knowing for whoever picks it up. ## Why this came up While grounding a Channels x AG-UI gap analysis, several `OSS-###` comments turned out to reference closed tickets — three canceled, two already done. The done ones (OSS-450 Teams managed, OSS-491 delivery terminal-signal) had already been cleaned out of `main` by subsequent work; these three canceled ones are what remain. ## Scope / notes - **Comment-only. No behavior change.** `oxfmt --check` and `oxlint` both clean. - Out of scope, flagged for a possible follow-up: `OSS-406` (`channels-intelligence/src/index.ts`) and `OSS-599` (`runtime/.../channel-manager.ts`, 5 references) are also canceled, but sit outside the channels packages surveyed here. Other referenced tickets (`OSS-96/162/476/566/568/621/623/641/670/691`) were not status-checked.