refactor: remove docsDir from InitTemplateConfig, use paths.docs only

Since the feature hasn't shipped yet, there's no need for backward
compatibility with docsDir in templates. This eliminates the dual-schema
confusion and simplifies the fallback chain.

Author: sotatek

packages/cli/src/__tests__/lib/InitTemplate.test.ts

@@ -88,22 +88,6 @@ environments:
     );
   });
 
-  it('loads template with custom docsDir', async () => {
-    mockFs.pathExists.mockResolvedValue(true as never);
-    mockFs.readFile.mockResolvedValue(`
-docsDir: .ai-docs
-environments:
-  - claude
-phases:
-  - requirements
-` as never);
-
-    const result = await loadInitTemplate('/tmp/init.yaml');
-
-    expect(result.docsDir).toBe('.ai-docs');
-    expect(result.environments).toEqual(['claude']);
-  });
-
   it('loads template with paths.docs config', async () => {
     mockFs.pathExists.mockResolvedValue(true as never);
     mockFs.readFile.mockResolvedValue(`
@@ -142,23 +126,14 @@ paths:
     );
   });
 
-  it('throws when docsDir is empty string', async () => {
+  it('rejects docsDir as unknown field', async () => {
     mockFs.pathExists.mockResolvedValue(true as never);
     mockFs.readFile.mockResolvedValue(`
-docsDir: "  "
+docsDir: .ai-docs
 ` as never);
 
     await expect(loadInitTemplate('/tmp/init.yaml')).rejects.toThrow(
-      '"docsDir" must be a non-empty string'
-    );
-  });
-
-  it('throws when docsDir is not a string', async () => {
-    mockFs.pathExists.mockResolvedValue(true as never);
-    mockFs.readFile.mockResolvedValue(JSON.stringify({ docsDir: 123 }) as never);
-
-    await expect(loadInitTemplate('/tmp/init.json')).rejects.toThrow(
-      '"docsDir" must be a non-empty string'
+      'unknown field(s): docsDir'
     );
   });
 

packages/cli/src/commands/init.ts

@@ -214,8 +214,6 @@ export async function initCommand(options: InitOptions) {
     docsDir = options.docsDir.trim();
   } else if (templateConfig?.paths?.docs) {
     docsDir = templateConfig.paths.docs;
-  } else if (templateConfig?.docsDir) {
-    docsDir = templateConfig.docsDir;
   }
 
   const phaseTemplateManager = new TemplateManager({ docsDir });

packages/cli/src/lib/InitTemplate.ts

@@ -11,7 +11,6 @@ export interface InitTemplateSkill {
 
 export interface InitTemplateConfig {
   version?: number | string;
-  docsDir?: string;
   paths?: {
     docs?: string;
   };
@@ -20,7 +19,7 @@ export interface InitTemplateConfig {
   skills?: InitTemplateSkill[];
 }
 
-const ALLOWED_TEMPLATE_FIELDS = new Set(['version', 'docsDir', 'paths', 'environments', 'phases', 'skills']);
+const ALLOWED_TEMPLATE_FIELDS = new Set(['version', 'paths', 'environments', 'phases', 'skills']);
 
 function validationError(templatePath: string, message: string): Error {
   return new Error(`Invalid template at ${templatePath}: ${message}`);
@@ -98,15 +97,6 @@ function validateTemplate(raw: unknown, resolvedPath: string): InitTemplateConfi
     }
   }
 
-  if (candidate.docsDir !== undefined) {
-    if (typeof candidate.docsDir !== 'string' || candidate.docsDir.trim().length === 0) {
-      throw validationError(resolvedPath, '"docsDir" must be a non-empty string');
-    }
-    if (!result.paths?.docs) {
-      result.docsDir = candidate.docsDir.trim();
-    }
-  }
-
   if (candidate.environments !== undefined) {
     if (!Array.isArray(candidate.environments)) {
       throw validationError(resolvedPath, '"environments" must be an array of environment codes');