Promote test → main (includes #447 segments-surface removal)#448
Conversation
Deletes GET /api/artist/segments, GET /api/chats/[id]/segment, the
create_segments MCP tool registration, and the lib/segments,
lib/artist/*segments*, lib/chats/getChatSegmentHandler, and
lib/supabase/{artist_segments,fan_segments,segments,segment_rooms}
modules. Also prunes the two remaining tool-chain references to the
create_segments tool in lib/chat/toolChains/{toolChains.ts,
createNewArtistToolChain.ts} so the orchestration does not reference
a tool that no longer exists.
The Segments feature is being removed entirely; the earlier migration
plan is abandoned. Part of SEGMENTS_SURFACE_REMOVAL_PLAN.md.
Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
Warning Rate limit exceeded
Your organization is not enrolled in usage-based pricing. Contact your admin to enable usage-based pricing to continue reviews beyond the rate limit, or try again in 38 minutes and 52 seconds. ⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. ℹ️ Review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (25)
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
No issues found across 26 files
Confidence score: 5/5
- Automated review surfaced no issues in the provided summaries.
- No files require special attention.
Requires human review: Large-scale removal of the 'Segments' feature, including multiple API endpoints and toolchain logic, requires human verification to confirm no active dependencies remain.
Architecture diagram
sequenceDiagram
participant Client as Client / UI
participant API as Next.js API Routes
participant MCP as MCP Tool Server
participant LLM as Language Model (LLM)
participant DB as Supabase Database
Note over Client,DB: Flow 1: New Artist Creation (Tool Chain)
MCP->>MCP: generate_txt_file (Arweave)
Note right of MCP: CHANGED: createNewArtistToolChain pruned
rect rgb(240, 240, 240)
Note over MCP,DB: REMOVED: create_segments tool execution
MCP--xLLM: analyze fan data & generate names
MCP--xDB: insert segments, artist_segments, & fan_segments
end
MCP->>MCP: NEW: Proceed directly to youtube_login
Note over Client,DB: Flow 2: Data Retrieval (Pruned Surface)
Client->>API: GET /api/artist/segments
API-->>Client: NEW: 404 Not Found (Route Deleted)
Client->>API: GET /api/chats/[id]/segment
API-->>Client: NEW: 404 Not Found (Route Deleted)
Note over Client,DB: Flow 3: AI Tooling (MCP Registration)
MCP->>MCP: registerAllTools()
Note right of MCP: REMOVED: registerCreateSegmentsTool
rect rgb(240, 240, 240)
Note over MCP,DB: Deleted Data Helpers
Note right of DB: lib/supabase/segments/* (Deleted)
Note right of DB: lib/supabase/fan_segments/* (Deleted)
Note right of DB: lib/supabase/artist_segments/* (Deleted)
Note right of DB: lib/supabase/segment_rooms/* (Deleted)
end
Promotes `test` → `main`.
Includes
All checks passed on the originating PR.
🤖 Generated with Claude Code
Summary by cubic
Removes the Segments feature across the codebase, deleting dead endpoints, tools, and data helpers. This prevents calls to non-existent segment APIs and the
create_segmentstool, and simplifies the codebase.Refactors
GET /api/artist/segments,GET /api/chats/:id/segment.create_segmentstool and registration; pruned references inlib/chat/toolChains/*andlib/mcp/tools/index.ts.lib/segments/*,lib/artist/*segments*,lib/chats/getChatSegmentHandler, andlib/supabase/{artist_segments,fan_segments,segments,segment_rooms}.Migration
GET /api/artist/segmentsandGET /api/chats/:id/segment; these routes no longer exist.create_segmentstool in any automations or prompts; it has been removed.Written for commit 83a58c8. Summary will update on new commits.