Refactor devlog tools and remove deprecated chat tools

- Removed deprecated `update_ai_context` tool from `ai-context-tools.ts`.
- Deleted `chat-tools.ts` file and its associated tools.
- Introduced new `devlog-tools.ts` file containing CRUD operations for devlog entries.
- Updated `index.ts` to include `devlogTools` and removed references to `progressTools` and `chatTools`.
- Removed `progress-tools.ts` as its functionality has been integrated into `devlog-tools.ts`.
- Updated project tools to reflect changes in project last accessed date.

Author: Marvin Zhang

.github/copilot-instructions.md

@@ -1,6 +1,6 @@
 # Devlog Project - Copilot Instructions
 
-## 🚨 CRITICAL RULES (READ FIRST!)
+<!-- ## 🚨 CRITICAL RULES (READ FIRST!)
 
 ### ⚠️ MANDATORY FIRST STEP: Always Discover Before Creating
 
@@ -20,7 +20,7 @@
 2. **📖 REVIEW**: Analyze discovered entries to understand context and avoid overlaps
 3. **✅ CREATE ONLY IF NEEDED**: Create new devlog entry using MCP tools only if no overlapping work exists
 4. **📝 TRACK PROGRESS**: Update entries with notes and status changes via MCP functions
-5. **🔗 LINK WORK**: Reference devlog IDs in commits and documentation
+5. **🔗 LINK WORK**: Reference devlog IDs in commits and documentation -->
 
 ## Development Philosophy (Brief)
 
@@ -50,107 +50,6 @@ import { DevlogEntry } from '@/types';                          // Ambiguous sel
 import { StorageProvider } from '../storage/providers';         // Missing .js extension
 ```
 
-## ⚠️ AUTOMATIC MIGRATION DETECTION
-
-### 🚨 **Contextual Migration Prompts**
-**These prompts trigger automatically based on file patterns being edited:**
-
-#### **When editing `packages/core/src/managers/**/*.ts`:**
-```
-⚠️  MIGRATION CHECK: Manager class changes often affect:
-- MCP adapter (packages/mcp/src/mcp-adapter.ts)
-- Web API routes (packages/web/app/api/*/route.ts)  
-- Web contexts (packages/web/app/contexts/*.tsx)
-
-Run: grep -r "ManagerClassName" packages/ --include="*.ts"
-```
-
-#### **When editing `packages/core/src/types/**/*.ts`:**
-```
-⚠️  MIGRATION CHECK: Type changes often affect:
-- All packages importing these types
-- MCP tool parameter validation
-- Web component prop types
-
-Run: grep -r "TypeName" packages/ --include="*.ts" --include="*.tsx"
-```
-
-#### **When editing `packages/core/src/storage/**/*.ts`:**
-```
-⚠️  MIGRATION CHECK: Storage changes often affect:
-- MCP storage tools (packages/mcp/src/tools/*)
-- Web API endpoints (packages/web/app/api/*)
-- Configuration and initialization code
-
-Run: grep -r "StorageInterface" packages/ --include="*.ts"
-```
-
-#### **When making any @codervisor/devlog-core changes:**
-```
-⚠️  AUTO-CHECK: After core changes, verify:
-1. pnpm detect-migration  # Automatic migration detection
-2. pnpm --filter @codervisor/devlog-mcp build
-3. pnpm --filter @codervisor/devlog-web build:test  
-4. Check for new compilation errors in dependent packages
-```
-
-#### **Before starting any development session:**
-```
-💡 PROACTIVE CHECK: Run automatic migration detection:
-pnpm detect-migration
-
-This scans recent changes and identifies potential cross-package impacts
-that require migration attention.
-```
-
-## 🔄 Architecture Migration Protocol
-
-### ⚠️ CRITICAL: For Major Architecture Changes
-
-**When modifying core classes, interfaces, or manager patterns:**
-
-### **MANDATORY Migration Steps:**
-1. **🔍 IMPACT ANALYSIS FIRST**: Map all dependent packages and components
-   - Search for usage across ALL packages (`grep -r "ClassName" packages/`)
-   - Identify adapters, API routes, tests, and integration points
-   - Document breaking changes and compatibility requirements
-
-2. **📋 MIGRATION PLAN**: Create systematic update sequence
-   - **Order**: Core → MCP → AI → Web (follow dependency chain)
-   - **Validation**: Build and test each package after updates
-   - **Integration**: Test cross-package functionality
-
-3. **✅ VALIDATION STRATEGY**: Define success criteria and checkpoints
-   - All packages build successfully
-   - All tests pass
-   - Integration workflows function correctly
-   - No runtime errors in dependent packages
-
-4. **🔙 ROLLBACK PLAN**: Document reversion procedures
-   - Backup state before migration
-   - Clear steps to restore previous version
-   - Dependency version compatibility matrix
-
-### **Cross-Package Dependency Map:**
-- **@codervisor/devlog-core changes** → ALWAYS update @codervisor/devlog-mcp, @codervisor/devlog-web APIs
-- **Manager class changes** → Update adapters, API routes, tests, web contexts
-- **Type/Interface changes** → Update ALL imports and usages across packages
-- **Storage provider changes** → Update web API endpoints and MCP tools
-
-### **Migration Validation Checklist:**
-```markdown
-□ Impact analysis completed across all packages
-□ Migration plan documented with specific steps
-□ Core package updated and builds successfully
-□ MCP package updated and builds successfully  
-□ Web package updated and builds successfully
-□ AI package updated and builds successfully (if affected)
-□ All cross-package tests pass
-□ Integration workflows validated
-□ Rollback procedure documented and tested
-□ Related devlog entries updated with migration status
-```
-
 ## Essential SOPs
 
 ### Documentation Synchronization Process

packages/core/src/types/requests.ts

@@ -27,7 +27,7 @@ export interface CreateDevlogRequest {
 }
 
 export interface UpdateDevlogRequest {
-  id: DevlogId;
+  id?: DevlogId;
   title?: string;
   description?: string;
   type?: DevlogType;

packages/mcp/src/adapters/adapter-factory.ts

@@ -1,9 +1,8 @@
 /**
- * Factory for creating MCP adapters based on configuration
- * Supports both direct core access and HTTP API client modes
+ * Factory for creating MCP adapters
+ * Uses HTTP API client for secure and isolated access to devlog operations
  */
 
-import { MCPDevlogAdapter } from './mcp-adapter.js';
 import { MCPApiAdapter, type MCPApiAdapterConfig } from './mcp-api-adapter.js';
 import {
   loadMCPConfig,
@@ -12,10 +11,10 @@ import {
   type MCPServerConfig,
 } from '../config/mcp-config.js';
 
-export type MCPAdapter = MCPDevlogAdapter | MCPApiAdapter;
+export type MCPAdapter = MCPApiAdapter;
 
 /**
- * Create an MCP adapter based on configuration
+ * Create an MCP adapter using HTTP API client
  */
 export async function createMCPAdapter(config?: MCPServerConfig): Promise<MCPAdapter> {
   // Load configuration if not provided
@@ -27,33 +26,19 @@ export async function createMCPAdapter(config?: MCPServerConfig): Promise<MCPAda
   // Print configuration summary for debugging
   printConfigSummary(mcpConfig);
 
-  let adapter: MCPAdapter;
+  // Create API-based adapter
+  const apiConfig: MCPApiAdapterConfig = {
+    apiClient: {
+      baseUrl: mcpConfig.webApi.baseUrl,
+      timeout: mcpConfig.webApi.timeout,
+      retries: mcpConfig.webApi.retries,
+    },
+    defaultProjectId: mcpConfig.defaultProjectId,
+    autoDiscoverWebService: mcpConfig.webApi.autoDiscover,
+  };
 
-  if (mcpConfig.mode === 'api') {
-    // Create API-based adapter
-    if (!mcpConfig.webApi) {
-      throw new Error('Web API configuration is required for API mode');
-    }
-
-    const apiConfig: MCPApiAdapterConfig = {
-      apiClient: {
-        baseUrl: mcpConfig.webApi.baseUrl,
-        timeout: mcpConfig.webApi.timeout,
-        retries: mcpConfig.webApi.retries,
-      },
-      defaultWorkspaceId: mcpConfig.defaultWorkspaceId,
-      autoDiscoverWebService: mcpConfig.webApi.autoDiscover,
-    };
-
-    adapter = new MCPApiAdapter(apiConfig);
-    console.log('Created MCP API Adapter (HTTP client mode)');
-  } else {
-    // Create direct core access adapter (existing implementation)
-    const directConfig = mcpConfig.direct || {};
-
-    adapter = new MCPDevlogAdapter(mcpConfig.defaultWorkspaceId);
-    console.log('Created MCP Direct Adapter (core access mode)');
-  }
+  const adapter = new MCPApiAdapter(apiConfig);
+  console.log('Created MCP API Adapter (HTTP client mode)');
 
   // Initialize the adapter
   await adapter.initialize();
@@ -66,7 +51,7 @@ export async function createMCPAdapter(config?: MCPServerConfig): Promise<MCPAda
  */
 export async function checkWebApiAvailability(baseUrl: string): Promise<boolean> {
   try {
-    const response = await fetch(`${baseUrl}/api/workspaces`, {
+    const response = await fetch(`${baseUrl}/api/projects`, {
       method: 'GET',
       headers: { 'Content-Type': 'application/json' },
       signal: AbortSignal.timeout(5000), // 5 second timeout
@@ -106,39 +91,24 @@ export async function discoverWebApiUrl(): Promise<string | null> {
 export async function createMCPAdapterWithDiscovery(): Promise<MCPAdapter> {
   const config = loadMCPConfig();
 
-  // If in API mode and auto-discovery is enabled, try to find web API
-  if (config.mode === 'api' && config.webApi?.autoDiscover) {
+  // If auto-discovery is enabled, try to find web API
+  if (config.webApi.autoDiscover) {
     const discoveredUrl = await discoverWebApiUrl();
 
     if (discoveredUrl) {
       // Update config with discovered URL
       config.webApi.baseUrl = discoveredUrl;
       console.log(`Using discovered web API URL: ${discoveredUrl}`);
     } else {
-      console.warn('Could not discover web API, falling back to direct mode');
-      config.mode = 'direct';
+      console.warn('Could not discover web API, using configured URL:', config.webApi.baseUrl);
     }
   }
 
   return createMCPAdapter(config);
 }
 
 /**
- * Get adapter interface type for tools
- */
-export function getAdapterInterface(adapter: MCPAdapter): 'direct' | 'api' {
-  return adapter instanceof MCPApiAdapter ? 'api' : 'direct';
-}
-
-/**
- * Type guard for direct adapter
- */
-export function isDirectAdapter(adapter: MCPAdapter): adapter is MCPDevlogAdapter {
-  return adapter instanceof MCPDevlogAdapter;
-}
-
-/**
- * Type guard for API adapter
+ * Type guard for API adapter (always true now)
  */
 export function isApiAdapter(adapter: MCPAdapter): adapter is MCPApiAdapter {
   return adapter instanceof MCPApiAdapter;

packages/mcp/src/adapters/index.ts

@@ -2,15 +2,12 @@
  * Adapter exports for MCP package
  */
 
-export { MCPDevlogAdapter } from './mcp-adapter.js';
 export { MCPApiAdapter } from './mcp-api-adapter.js';
 export {
   createMCPAdapter,
   createMCPAdapterWithDiscovery,
   checkWebApiAvailability,
   discoverWebApiUrl,
-  getAdapterInterface,
-  isDirectAdapter,
   isApiAdapter,
   type MCPAdapter,
 } from './adapter-factory.js';