fix: address review feedback from CodeRabbit and Devin

- Include test step in no-checklist fallback when includeTests is true
- Restore observations from snapshot in SessionSnapshotRestoreCommand

Author: rohitg00

packages/cli/src/commands/session.ts

@@ -402,7 +402,7 @@ export class SessionSnapshotRestoreCommand extends Command {
       return 1;
     }
 
-    const { sessionState } = manager.restore(this.name);
+    const { sessionState, observations } = manager.restore(this.name);
 
     const sessionMgr = new SessionManager(projectPath);
     const currentState = sessionMgr.getOrCreate();
@@ -413,6 +413,22 @@ export class SessionSnapshotRestoreCommand extends Command {
     });
     sessionMgr.save();
 
+    try {
+      if (observations && observations.length > 0) {
+        const store = new ObservationStore(projectPath);
+        for (const obs of observations) {
+          store.add(
+            obs.type as Parameters<ObservationStore['add']>[0],
+            obs.content as unknown as Parameters<ObservationStore['add']>[1],
+            obs.agent as Parameters<ObservationStore['add']>[2],
+            obs.relevance,
+          );
+        }
+      }
+    } catch {
+      // Non-critical: session state restored even if observations fail
+    }
+
     console.log(chalk.green(`\u2713 Snapshot restored: ${this.name}`));
     return 0;
   }

packages/core/src/plan/issue-planner.ts

@@ -228,6 +228,9 @@ export class IssuePlanner {
         files,
         steps: [
           { type: 'implement', description: issue.title },
+          ...(includeTests
+            ? [{ type: 'test' as const, description: `Write tests for ${issue.title}` }]
+            : []),
         ],
       });
     }