fix(config): handle null rules field in project config

src/core/project-config.ts

@@ -117,8 +117,8 @@ export function readProjectConfig(projectRoot: string): ProjectConfig | null {
     if (raw.rules !== undefined) {
       const rulesField = z.record(z.string(), z.array(z.string()));
 
-      // First check if it's an object structure
-      if (typeof raw.rules === 'object' && !Array.isArray(raw.rules)) {
+      // First check if it's an object structure (guard against null since typeof null === 'object')
+      if (typeof raw.rules === 'object' && raw.rules !== null && !Array.isArray(raw.rules)) {
         const parsedRules: Record<string, string[]> = {};
         let hasValidRules = false;
 

test/core/project-config.test.ts

@@ -142,6 +142,30 @@ rules: ["not", "an", "object"]
         );
       });
 
+      it('should handle rules: null without aborting config parsing', () => {
+        // YAML `rules:` with no value parses to null
+        const configDir = path.join(tempDir, 'openspec');
+        fs.mkdirSync(configDir, { recursive: true });
+        fs.writeFileSync(
+          path.join(configDir, 'config.yaml'),
+          `schema: spec-driven
+context: Valid context
+rules:
+`
+        );
+
+        const config = readProjectConfig(tempDir);
+
+        // Should still parse schema and context despite null rules
+        expect(config).toEqual({
+          schema: 'spec-driven',
+          context: 'Valid context',
+        });
+        expect(consoleWarnSpy).toHaveBeenCalledWith(
+          expect.stringContaining("Invalid 'rules' field")
+        );
+      });
+
       it('should filter out invalid rules for specific artifact', () => {
         const configDir = path.join(tempDir, 'openspec');
         fs.mkdirSync(configDir, { recursive: true });