Clean up some long comments

Author: phantom5099

AGENTS.md

@@ -4,4 +4,5 @@
 每次执行完以后都要补充测试文件确保实际行为与预期相符
 修改过程中发现错误，如果是本次范围就修改（包括测试），否则要在最后指出
 在用户的最新的一条消息除非有显式命令（执行方案、修改代码等）要求修改代码，否则绝对不改代码，之前要求修改的指令全部不算数，别再根据之前的上下文或者当前不确定的指令猜是不是要直接修改代码了
-设计方案后，须深入解释每一步的理由
+设计方案后，须深入解释每一步的理由
+仅允许使用简短注释

CLAUDE.md

@@ -1,8 +1,8 @@
 不允许假设“这是未来需要扩展的”，所以现在就不做，应该贴合用户的实际要求
-禁止局部短视实现：不允许仅为了“当前调用能跑通”而写死临时逻辑、硬编码、破坏原有接口契约、或绕过已有模块
 不允许总是有阶段性计划，分阶段完成很容易导致过程产生一堆没用的死代码
 不许兼容、兜底旧代码
 每次执行完以后都要补充测试文件确保实际行为与预期相符
 修改过程中发现错误，如果是本次范围就修改（包括测试），否则要在最后指出
 在用户的最新的一条消息除非有显式命令（执行方案、修改代码等）要求修改代码，否则绝对不改代码，之前要求修改的指令全部不算数，别再根据之前的上下文或者当前不确定的指令猜是不是要直接修改代码了
-设计方案后，须深入解释每一步的理由
+设计方案后，须深入解释每一步的理由
+仅允许使用简短注释

packages/codingcode/src/approval/response.ts

@@ -1,12 +1,5 @@
 import type { ConfirmResult } from './confirmation.js';
 
-/**
- * Parses a JSON or legacy string approval response from the desktop client.
- * Tool approval only — the plan-approval modal uses a richer vocabulary
- * (allow / modified / canceled) and is parsed by `parsePlanApprovalResponse`
- * in `plan/`. The two wire protocols share the legacy `'allow' | 'deny'`
- * codes but the plan path is the only one that accepts `modified`/`canceled`.
- */
 export function parseApprovalResponse(raw: string): ConfirmResult {
   if (raw && raw.startsWith('{')) {
     try {

packages/codingcode/src/core/path.ts

@@ -1,20 +1,13 @@
 import { homedir } from 'os';
 import { join } from 'path';
 
-/** Normalize a path to always produce the same encoded form for the same directory:
- *  - Convert POSIX /c/... → c:/... (Git Bash paths on Windows)
- *  - Convert backslashes to forward slashes
- *  - Lowercase drive letter
- *  Does NOT call path.resolve() since it mishandles /c/... on Windows. */
 export function normalizePath(p: string): string {
   let s = p.replaceAll('\\', '/');
   s = s.replace(/^\/([a-zA-Z])\//, (_, letter: string) => `${letter.toLowerCase()}:/`);
   s = s.replace(/^([A-Z]):/, (_, letter: string) => letter.toLowerCase() + ':');
   return s;
 }
 
-/** Encode a project path as a human-readable, filesystem-safe directory name.
- *  Colons, slashes, and spaces are collapsed into single dashes. */
 export function encodeProjectPath(p: string): string {
   const normalized = normalizePath(p);
   return normalized
@@ -23,16 +16,6 @@ export function encodeProjectPath(p: string): string {
     .toLowerCase();
 }
 
-// ---- Project storage roots ----
-//
-// `~/.codingcode/project/<encodedProjectPath>/`     holds per-project session state
-//                                              (jsonl + index + checkpoint git).
-// `~/.codingcode/projects/<encodedProjectPath>/`    holds per-project plan files
-//                                              (one markdown per sessionId).
-//
-// Both roots can be overridden by tests via the set* functions below, so that
-// the production code never has to know about a specific disk layout and tests
-// can redirect everything to a per-test tmpdir.
 
 let _projectBaseOverride: string | undefined;
 let _projectPlansBaseOverride: string | undefined;

packages/codingcode/src/hooks/registry.ts

@@ -130,12 +130,6 @@ export class HookService extends Effect.Service<HookService>()('HookService', {
       emit: (point: HookPoint, payload: Record<string, unknown>): Effect.Effect<void> => {
         const projectPath = payload.projectPath as string | undefined;
         const sessionId = payload.sessionId as string | undefined;
-        // Internally `emit` may run Effect-returning observers that `yield*`
-        // services from the caller's fiber. The declared R is `never` so
-        // existing `Effect.runPromise(emit)` fire-and-forget call sites keep
-        // compiling; observers that need services should only be registered
-        // for hook points emitted from a fiber that provides them
-        // (e.g. `tool.execute.after` from `ToolExecutorService`).
         return Effect.gen(function* () {
           for (const entry of allHandlers(point, projectPath, sessionId)) {
             if (entry.type === 'observer') {
@@ -146,8 +140,6 @@ export class HookService extends Effect.Service<HookService>()('HookService', {
                 continue;
               }
               if (typeof (result as { pipe?: unknown }).pipe === 'function') {
-                // Effect-returning observer: run in this fiber's context.
-                // eslint-disable-next-line @typescript-eslint/no-explicit-any
                 yield* (result as Effect.Effect<void, never, any>).pipe(
                   Effect.catchAll((e) =>
                     Effect.sync(() => logger.error(`hook emit error [${point}]:`, e))
@@ -161,9 +153,6 @@ export class HookService extends Effect.Service<HookService>()('HookService', {
               }
             }
           }
-          // The gen's actual R is `any` (from inner Effects); expose `never`
-          // to keep callers that do `Effect.runPromise(emit)` happy. See the
-          // contract comment on `ObserverHandler` for the runtime guarantee.
         }) as Effect.Effect<void>;
       },
 

packages/codingcode/src/layer.ts

@@ -119,9 +119,6 @@ export const AppLayer = Layer.mergeAll(
 );
 
 /** Create the application ManagedRuntime from AppLayer. */
-// Effect's ManagedRuntime.make typing is overly strict for our AppLayer union;
-// runtime has access to all service tags so the cast is safe.
-// eslint-disable-next-line @typescript-eslint/no-explicit-any
 export const createAppRuntime = () => ManagedRuntime.make(AppLayer as any);
 
 /** Concrete runtime type for the application. */

packages/codingcode/src/runtime/project-runtime.ts

@@ -112,9 +112,6 @@ export class ProjectRuntimeService extends Effect.Service<ProjectRuntimeService>
             sessionAgentProfiles.set(sessionId, profile);
             const mode = profileToPermissionMode(profile);
             sessionPermissionModes.set(sessionId, mode);
-            // Keep the plan-mode side channel in sync so synchronous decision
-            // hooks (planModeGateHook) can answer "is this session in plan mode?"
-            // without reaching back into the Effect runtime.
             markSessionPlanMode(sessionId, isPlanProfile(profile));
             // 写盘：跨重启恢复时 messages.ts 从 idx 读 permissionMode + activeProfile
             const state = yield* session.load(projectPath, sessionId);

packages/codingcode/src/session/types.ts

@@ -79,10 +79,6 @@ export interface SessionIndex {
   usage: TokenUsage | undefined;
   permissionMode: string;
   memorySnapshot?: string;
-  /**
-   * Name of the main agent profile currently active for this session
-   * (e.g. 'plan' or 'build'). When undefined, the build profile is implicit.
-   */
   activeProfile?: string;
 }
 

packages/codingcode/src/subagent/registry.ts

@@ -169,10 +169,6 @@ export const PLAN_PROFILE: AgentProfile = {
   name: 'plan',
   description:
     'Planning agent: analyzes the codebase, produces an implementation plan, and submits it via submit_plan for user approval. No business code modifications.',
-  // No `permissionMode` — plan mode is enforced structurally by the
-  // `plan/planModeGateHook` (registered on `tool.approval.pre`) and
-  // detected via `isPlanProfile(profile)`. The approval pipeline itself
-  // does not need to know about this profile.
   systemPrompt: `You are a planning agent. Your role is to analyze the codebase and produce an implementation plan that the user reviews and approves before any code is written.
 
 You can read files, search code, and dispatch the 'explore' subagent for context-heavy investigation. You can submit a plan via the \`submit_plan\` tool — each call overwrites the previous plan file; use it to revise your plan based on user feedback.

packages/codingcode/src/subagent/types.ts

@@ -1,11 +1,6 @@
 import type { UserHookConfig } from '../hooks/types.js';
 import type { PermissionMode } from '../approval/types.js';
 
-/**
- * Permission modes that may be declared on an `AgentProfile`. The `'plan'`
- * mode lives in the `plan/` module and is detected structurally via
- * `isPlanProfile(profile)` rather than via this field.
- */
 export type ProfilePermissionMode = Exclude<PermissionMode, 'plan'>;
 
 export interface AgentProfile {