fix(stubs): remove hardcoded paths from StubLoader and trim flaky integration tests

- Remove FLUTTER_TEST branch with hardcoded /Users/anilcan/... fallback
  that fails on CI (Ubuntu). Keep clean 4-strategy resolution:
  env var → package_config.json → Platform.script walk-up → cwd fallback
- Add MAGIC_CLI_STUBS_DIR env var as first-priority override for tests
- Remove stub-dependent integration tests (make:*, install, --force)
  that duplicate unit test coverage and fail when Directory.current
  changes to tempDir. Keep Kernel dispatch + key:generate tests
- Add .claude/rules/ path-scoped convention files (dart, commands,
  tests, stubs)

Author: anilcancakir

.claude/rules/commands.md

@@ -0,0 +1,51 @@
+---
+path: "lib/src/commands/**/*.dart"
+---
+
+# Command Conventions
+
+## GeneratorCommand Contract
+Every `make:*` command extends `GeneratorCommand` and overrides:
+- `name` → `'make:feature'`
+- `description` → `'Create a new feature class'`
+- `getDefaultNamespace()` → output directory (e.g., `'lib/app/controllers'`)
+- `getStub()` → stub name without `.stub` extension
+- `getReplacements(String name)` → custom placeholder map
+- `getProjectRoot()` → `_testRoot ?? super.getProjectRoot()`
+
+## Constructor Pattern
+```dart
+final String? _testRoot;
+MakeXCommand({String? testRoot}) : _testRoot = testRoot;
+```
+Every command accepts optional `testRoot` for test isolation.
+
+## configure() Must Call Super
+```dart
+@override
+void configure(ArgParser parser) {
+  super.configure(parser);  // Inherits --force flag
+  parser.addFlag('custom', abbr: 'c', negatable: false);
+}
+```
+Always `super.configure(parser)` first — skipping loses `--force` inheritance.
+
+## Suffix Handling
+Commands that auto-append suffixes (Controller, Factory, Seeder, Provider, Request, Policy):
+- Strip existing suffix before re-appending to prevent `UserControllerController`
+- Use private `_resolveClassName()` or `_stripSuffix()` + `_withSuffix()` helpers
+
+## Command Chaining
+`MakeModelCommand` chains child generators via:
+```dart
+await MakeMigrationCommand(testRoot: _testRoot).runWith([migrationName, '--create=$tableName']);
+```
+Always pass `testRoot: _testRoot` to child commands — omitting causes tests to touch real filesystem.
+
+## Non-Generator Commands
+`InstallCommand` and `KeyGenerateCommand` extend `Command` directly (not `GeneratorCommand`).
+They handle file creation manually without stubs.
+
+## Nested Path Support
+`make:controller Admin/Dashboard` → `StringHelper.parseName()` returns `(directory: 'admin', className: 'Dashboard', fileName: 'dashboard')`.
+Directory segments are snake_cased. Class name preserves PascalCase.

.claude/rules/dart.md

@@ -0,0 +1,30 @@
+---
+path: "**/*.dart"
+---
+
+# Dart Conventions
+
+## Imports
+- Always `package:magic_cli/src/...` — never relative imports
+- Order: `dart:` stdlib → external packages (`args`, `path`, `yaml`) → internal `package:magic_cli/...`
+- Alias `package:path/path.dart` as `path`
+
+## Naming
+- Classes: `PascalCase` — Files: `snake_case` — Methods: `camelCase` — Private: `_prefix`
+- CLI command names: `namespace:action` (e.g., `make:controller`, `key:generate`)
+- One class per file, filename matches class in snake_case
+
+## Doc Comments
+- `///` on all public classes and methods
+- Class-level: usage examples in `## Usage` + `## Output` sections with code blocks
+- Method-level: single-line summary, then `@param`, `@return`, `@throws` as needed
+
+## Error Handling
+- File operations: throw `FileSystemException('message', path)`
+- Missing CLI args: print error via `error()` and return — do not throw
+- Kernel catches `FormatException` + generic `Exception`, sets `exitCode = 1`
+
+## Static Utility Classes
+- Constructor: `const ClassName._()` to prevent instantiation
+- All methods `static`, no instance state
+- Follows: `FileHelper`, `ConsoleStyle`, `ConfigEditor`, `JsonEditor`, `StringHelper`

.claude/rules/stubs.md

@@ -0,0 +1,33 @@
+---
+path: "assets/stubs/**/*.stub"
+---
+
+# Stub Template Conventions
+
+## Placeholder Syntax
+- Use `{{ key }}` with spaces inside braces — flexible whitespace (`{{key}}` also works)
+- Case-sensitive: `{{ className }}` only — `{{ classname }}` is a different placeholder
+- Unreplaced placeholders remain as raw text in output — no validation or warnings
+
+## Auto-Handled Placeholders
+- `{{ className }}` — PascalCase class name (from last path segment)
+- `{{ namespace }}` — output directory path (e.g., `lib/app/controllers`)
+
+## Custom Placeholders
+Defined per command via `getReplacements()`:
+- `{{ tableName }}` — plural snake_case (model)
+- `{{ snakeName }}` — snake_case of class name
+- `{{ resourceName }}` — lower-case resource name
+- `{{ description }}` — human-readable description
+
+## Naming Convention
+- `{feature}.stub` — default template (e.g., `controller.stub`)
+- `{feature}.{variant}.stub` — variant selected by flag (e.g., `controller.resource.stub`, `view.stateful.stub`, `migration.create.stub`)
+
+## Content Style
+- Start with `import 'package:magic/magic.dart';` (or relevant Magic imports)
+- Class docstring: `/// {{ className }} description.`
+- Private fields with `_prefix`
+- `// TODO:` comments for user customization points
+- Trailing commas on multi-line params
+- Match the coding style of the parent Magic framework

.claude/rules/tests.md

@@ -0,0 +1,61 @@
+---
+path: "test/**/*_test.dart"
+---
+
+# Test Conventions
+
+## Setup Pattern
+```dart
+late Directory tempDir;
+late MyCommand cmd;
+late ArgParser parser;
+
+setUp(() {
+  tempDir = Directory.systemTemp.createTempSync('magic_test_feature_');
+  cmd = MyCommand(testRoot: tempDir.path);
+  parser = ArgParser();
+  cmd.configure(parser);
+});
+
+tearDown(() {
+  tempDir.deleteSync(recursive: true);
+});
+```
+Every test group gets a fresh `tempDir`. Always clean up in `tearDown`.
+
+## Test Isolation
+- Pass `testRoot: tempDir.path` to ALL commands — never let tests touch real project
+- For chained commands (`make:model -mcf`), child commands must also receive `testRoot`
+- Set `MAGIC_CLI_STUBS_DIR` env var to override stub resolution paths in tests
+
+## Running Commands in Tests
+```dart
+cmd.arguments = parser.parse(['Monitor', '--resource']);
+await cmd.handle();
+```
+Always `parser.parse()` before `cmd.handle()` — populates `cmd.arguments`.
+
+## Assertions
+- File existence: `expect(File('...').existsSync(), isTrue)`
+- Content check: `expect(file.readAsStringSync(), contains('class MonitorController'))`
+- Negative: `expect(content, isNot(contains('MonitorControllerController')))`
+- Count: `expect(dir.listSync().length, equals(1))`
+
+## Test Structure Per Command
+1. Name/description getters
+2. Argument parsing (flags, options, positional args)
+3. File generation — verify file exists at correct path
+4. Content verification — class name, imports, placeholders replaced
+5. Edge cases — nested paths, duplicate suffixes, `--force` overwrite, missing args
+
+## Integration Tests
+`Directory.current` must be changed and restored in `try`/`finally`:
+```dart
+Directory.current = tempDir;
+try { await kernel.handle(args); }
+finally { Directory.current = originalDir; }
+```
+Create a dummy `pubspec.yaml` in `tempDir` — `findProjectRoot()` needs it.
+
+## No Mocking Libraries
+Use real filesystem with temp dirs. No `mockito`, no code generation — mock by constructing commands with `testRoot`.

lib/src/stubs/stub_loader.dart

@@ -76,35 +76,17 @@ class StubLoader {
   ///    `name: magic_cli` (works for direct script execution).
   /// 3. Check common development paths as fallback.
   static List<String> _defaultSearchPaths() {
-    // First, check if we are in a test environment. If we are running tests,
-    // we should use the current working directory's assets/stubs if it exists.
-    // In tests, the script path is often a temp file generated by the test runner.
-    if (Platform.environment.containsKey('FLUTTER_TEST') ||
-        Platform.script.path.endsWith('.dart') &&
-            Platform.script.path.contains('test')) {
-      final possibleRoots = [
-        Directory.current.path,
-        // When running test from magic root directory
-        path.join(Directory.current.path, 'plugins', 'magic_cli'),
-        // When running tests from IDE, Directory.current might be the project root
-        // but magic_cli is inside plugins
-        path.join(
-            Directory.current.path, 'plugins', 'magic', 'plugins', 'magic_cli'),
-        // The actual absolute path where we know it exists during development
-        '/Users/anilcan/StudioProjects/uptizm/plugins/magic/plugins/magic_cli'
-      ];
-
-      for (final root in possibleRoots) {
-        final stubDir = path.join(root, 'assets', 'stubs');
-        if (Directory(stubDir).existsSync()) {
-          return [stubDir];
-        }
-      }
+    // 1. Environment variable override — highest priority.
+    //    Used in tests: set MAGIC_CLI_STUBS_DIR to point at a custom stubs dir.
+    final envDir = Platform.environment['MAGIC_CLI_STUBS_DIR'];
+    if (envDir != null && Directory(envDir).existsSync()) {
+      return [envDir];
     }
 
-    // 1. Parse .dart_tool/package_config.json — the most reliable strategy.
+    // 2. Parse .dart_tool/package_config.json — the most reliable strategy.
     //    This file is generated by `dart pub get` and contains the rootUri
     //    for every dependency, including path dependencies.
+    //    Works in all environments: local dev, CI, test, consumer projects.
     final packageConfigRoot = _resolveFromPackageConfig();
     if (packageConfigRoot != null) {
       final stubDir = path.join(packageConfigRoot, 'assets', 'stubs');
@@ -113,7 +95,7 @@ class StubLoader {
       }
     }
 
-    // 2. Walk up from Platform.script to find the magic_cli pubspec.
+    // 3. Walk up from Platform.script to find the magic_cli pubspec.
     //    Works when running the CLI directly from its own directory.
     final scriptPath = Platform.script.toFilePath();
     var current = Directory(path.dirname(scriptPath));
@@ -131,12 +113,10 @@ class StubLoader {
       current = parent;
     }
 
-    // 3. Fallback: check common development paths.
+    // 4. Fallback: check current directory and common dev paths.
     final possibleRoots = [
-      path.join(Directory.current.path, 'plugins', 'magic_cli'),
       Directory.current.path,
-      // For testing, since tests run in temp directories
-      '/Users/anilcan/StudioProjects/uptizm/plugins/magic/plugins/magic_cli'
+      path.join(Directory.current.path, 'plugins', 'magic_cli'),
     ];
 
     for (final root in possibleRoots) {