refactor: remove MCP server references from documentation and codebase

Author: lucaforni

AGENTS.md

@@ -5,10 +5,9 @@ This file provides guidance for AI coding agents working on this codebase.
 ## Project Overview
 
 OpenCode Team Sync (oct) is a CLI tool for synchronizing OpenCode configurations
-(agents, skills, MCP servers) across teams via Git repositories. The project is
-currently in the planning/specification phase.
+(agents and skills) across teams via Git repositories.
 
-**Tech Stack**: TypeScript 5.x, Node.js 18+, Commander.js, Vitest, tsup
+**Tech Stack**: TypeScript 5.x, Node.js 18+, Commander.js, Vitest, tsup, minimatch
 
 ## Build & Development Commands
 
@@ -31,9 +30,9 @@ npm run format                                 # Formatting
 src/
 ├── cli/commands/        # init, sync, status, list, validate, update, rollback, remove, clean, info
 ├── core/                # discovery, git, sync, validator, namespace, lockfile, tags
-├── schemas/             # Zod schemas for agent, skill, mcp, manifest, lockfile
+├── schemas/             # Zod schemas for agent, skill, manifest, lockfile
 ├── types/               # TypeScript interfaces
-└── utils/               # fs-utils, path-utils, hash-utils, error-utils, logger
+└── utils/               # fs-utils, path-utils, hash-utils, error-utils, logger, yaml-utils
 
 tests/
 ├── unit/                # Fast, isolated tests (~70%)
@@ -120,10 +119,11 @@ export type AgentFrontmatter = z.infer<typeof AgentFrontmatterSchema>;
 ```typescript
 interface ConfigEntry {
   path: string;           // Relative path in repo
-  type: ConfigType;       // 'agent' | 'skill' | 'mcp'
+  type: ConfigType;       // 'agent' | 'skill'
   name: string;           // Derived from filename/directory
   hash: string;           // SHA-256 of content
   tags: string[];
+  description?: string;   // Optional metadata
 }
 
 interface SyncResult {
@@ -138,8 +138,14 @@ interface SyncResult {
 ## Configuration Types
 
 - **Agents**: Markdown files with YAML frontmatter (`.md`)
+  - Patterns: `agents/**/*.md`, `*.agent.md`
+  - Validated by: `AgentValidator`
 - **Skills**: SKILL.md files in skill directories
-- **MCP Servers**: JSON/YAML configuration files
+  - Patterns: `skills/**/SKILL.md`, `skill/**/SKILL.md`
+  - Validated by: `SkillValidator`
+
+**Note**: MCP servers are NOT distributed as files. They are configured in 
+`opencode.json` and should be documented in team repos but not synced.
 
 ## Namespace Isolation
 

ARCHITECTURE.md

@@ -361,7 +361,6 @@ src/
 │   │   ├── validation-engine.ts
 │   │   ├── agent-validator.ts
 │   │   ├── skill-validator.ts
-│   │   ├── mcp-validator.ts
 │   │   └── manifest-validator.ts
 │   ├── namespace/
 │   │   ├── namespace-manager.ts
@@ -376,7 +375,6 @@ src/
 ├── schemas/               # Zod schemas
 │   ├── agent.schema.ts
 │   ├── skill.schema.ts
-│   ├── mcp.schema.ts
 │   ├── manifest.schema.ts
 │   └── lockfile.schema.ts
 │
@@ -509,11 +507,6 @@ export class DiscoveryEngine {
    */
   private async discoverSkills(path: string): Promise<ConfigEntry[]>
 
-  /**
-   * Discover MCP servers (.json/.yaml in mcp/)
-   */
-  private async discoverMCP(path: string): Promise<ConfigEntry[]>
-
   /**
    * Parse and merge with manifest if present
    */
@@ -860,15 +853,15 @@ function matchTags(configTags: string[], filterTags: string[]): boolean {
  */
 export interface ConfigEntry {
   path: string                    // Relative path in repo
-  type: ConfigType                // agent | skill | mcp
+  type: ConfigType                // agent | skill
   name: string                    // Derived from filename/directory
   hash: string                    // SHA-256 hash of content
   tags: string[]                  // Associated tags
   content?: string                // Actual file content
   syncedAt?: string              // ISO timestamp
 }
 
-export type ConfigType = 'agent' | 'skill' | 'mcp'
+export type ConfigType = 'agent' | 'skill'
 
 /**
  * Lockfile structure
@@ -965,35 +958,6 @@ export const SkillFrontmatterSchema = z.object({
   compatibility: z.string().optional(),
   metadata: z.record(z.string()).optional()
 })
-
-/**
- * MCP local server schema
- */
-export const MCPLocalSchema = z.object({
-  type: z.literal('local'),
-  command: z.array(z.string()),
-  environment: z.record(z.string()).optional(),
-  enabled: z.boolean().optional(),
-  timeout: z.number().optional()
-})
-
-/**
- * MCP remote server schema
- */
-export const MCPRemoteSchema = z.object({
-  type: z.literal('remote'),
-  url: z.string().url(),
-  enabled: z.boolean().optional(),
-  headers: z.record(z.string()).optional(),
-  oauth: z.union([z.boolean(), z.object({
-    clientId: z.string().optional(),
-    clientSecret: z.string().optional(),
-    scope: z.string().optional()
-  })]).optional(),
-  timeout: z.number().optional()
-})
-
-export const MCPSchema = z.union([MCPLocalSchema, MCPRemoteSchema])
 ```
 
 ---
@@ -1183,8 +1147,7 @@ describe('Init and Sync Integration', () => {
   beforeEach(async () => {
     testRepo = await createTestRepository({
       agents: ['frontend-dev.md', 'backend-api.md'],
-      skills: ['git-release/SKILL.md'],
-      mcp: ['jira.json']
+      skills: ['git-release/SKILL.md']
     })
     tempDir = await createTempDir()
   })

ROADMAP.md

@@ -253,7 +253,6 @@
 - [ ] Define Zod schemas (3h)
   - Agent schema (frontmatter validation)
   - Skill schema (SKILL.md validation)
-  - MCP schema (JSON validation)
   - Manifest schema (.opencode-team.yaml)
   - Lockfile schema
 
@@ -520,7 +519,7 @@
 **Subtasks**:
 - [ ] List functionality (3h)
   - List all configs (team + personal)
-  - Filter by type (agent/skill/mcp)
+  - Filter by type (agent/skill)
   - Filter by tags
   - Sort options
 
@@ -594,7 +593,7 @@
   ```bash
   oct remove <config-name> [options]
   Options:
-    --type <type>    Config type (agent/skill/mcp)
+    --type <type>    Config type (agent/skill)
   ```
   - Remove single config
   - Update lockfile