@aliaksei-raketski/pi-angular-developer 0.1.0

package/skills/angular-developer/references/signals-overview.md

@@ -0,0 +1,94 @@
+# Angular Signals Overview
+
+Signals are the foundation of reactivity in modern Angular applications. A **signal** is a wrapper around a value that notifies interested consumers when that value changes.
+
+## Writable Signals (`signal`)
+
+Use `signal()` to create state that can be directly updated.
+
+```ts
+import {signal} from '@angular/core';
+
+// Create a writable signal
+const count = signal(0);
+
+// Read the value (always requires calling the getter function)
+console.log(count());
+
+// Update the value directly
+count.set(3);
+
+// Update based on the previous value
+count.update((value) => value + 1);
+```
+
+### Exposing as Readonly
+
+When exposing state from a service, it is a best practice to expose a readonly version to prevent external mutation.
+
+```ts
+private readonly _count = signal(0);
+// Consumers can read this, but cannot call .set() or .update()
+readonly count = this._count.asReadonly();
+```
+
+## Computed Signals (`computed`)
+
+Use `computed()` to create read-only signals that derive their value from other signals.
+
+- **Lazily Evaluated**: The derivation function doesn't run until the computed signal is read.
+- **Memoized**: The result is cached. It only recalculates when one of the signals it depends on changes.
+- **Dynamic Dependencies**: Only the signals _actually read_ during the derivation are tracked.
+
+```ts
+import {signal, computed} from '@angular/core';
+
+const count = signal(0);
+const doubleCount = computed(() => count() * 2);
+
+// doubleCount automatically updates when count changes.
+```
+
+## Reactive Contexts
+
+A **reactive context** is a runtime state where Angular monitors signal reads to establish a dependency.
+
+Angular automatically enters a reactive context when evaluating:
+
+- `computed` signals
+- `effect` callbacks
+- `linkedSignal` computations
+- Component templates
+
+### Untracked Reads (`untracked`)
+
+If you need to read a signal inside a reactive context _without_ creating a dependency (so that the context doesn't re-run when the signal changes), use `untracked()`.
+
+```ts
+import {effect, untracked} from '@angular/core';
+
+effect(() => {
+  // This effect only runs when currentUser changes.
+  // It does NOT run when counter changes, even though counter is read here.
+  console.log(`User: ${currentUser()}, Count: ${untracked(counter)}`);
+});
+```
+
+### Async Operations in Reactive Contexts
+
+The reactive context is only active for **synchronous** code. Signal reads after an `await` will not be tracked. **Always read signals before asynchronous boundaries.**
+
+```ts
+// ❌ INCORRECT: theme() is not tracked because it is read after await
+effect(async () => {
+  const data = await fetchUserData();
+  console.log(theme());
+});
+
+// ✅ CORRECT: Read the signal before the await
+effect(async () => {
+  const currentTheme = theme();
+  const data = await fetchUserData();
+  console.log(currentTheme);
+});
+```

package/skills/angular-developer/references/tailwind-css.md

@@ -0,0 +1,69 @@
+# Using Tailwind CSS with Angular
+
+Tailwind CSS is a utility-first CSS framework that integrates seamlessly with Angular.
+
+**CRITICAL AGENT GUIDANCE: ALWAYS focus on Tailwind CSS v4 practices. DO NOT revert to old Tailwind v3 patterns (like creating `tailwind.config.js` with `@tailwind` directives) as this will break the application build. Modern Angular projects use Tailwind v4.**
+
+## Automated Setup (Recommended)
+
+The easiest way to add Tailwind CSS to an Angular project is via the Angular CLI:
+
+```shell
+ng add tailwindcss
+```
+
+This will automatically:
+
+1. Install `tailwindcss` and peer dependencies.
+2. Configure the project to use Tailwind CSS.
+3. Add the proper import to your global styles.
+
+## Manual Setup (Tailwind v4)
+
+If setting up manually, use the following Tailwind v4 pattern:
+
+### 1. Install Dependencies
+
+Install Tailwind CSS and PostCSS:
+
+```shell
+npm install tailwindcss @tailwindcss/postcss postcss
+```
+
+### 2. Configure PostCSS
+
+Create a `.postcssrc.json` file in the project root:
+
+```json
+{
+  "plugins": {
+    "@tailwindcss/postcss": {}
+  }
+}
+```
+
+_(Do NOT create a `tailwind.config.js` file! Configuration in v4 is handled through CSS variables)._
+
+### 3. Import Tailwind CSS
+
+In your global styles file (e.g., `src/styles.css`), add the standard v4 import:
+
+```css
+@import 'tailwindcss';
+```
+
+_(If using SCSS, use `@use 'tailwindcss';` instead)._
+
+### 4. Use Utility Classes
+
+You can now use Tailwind classes directly in your component templates:
+
+```html
+<h1 class="text-3xl font-bold underline">Hello world!</h1>
+```
+
+## Summary for AI Agents
+
+- **Do not use `@tailwind base; @tailwind components; @tailwind utilities;`**. Use `@import 'tailwindcss';`.
+- **Do not create `tailwind.config.js`**. Configuration is managed directly in CSS via theme variables or using PostCSS configurations.
+- Stick strictly to v4 syntax and workflows.

package/skills/angular-developer/references/template-driven-forms.md

@@ -0,0 +1,114 @@
+# Template-Driven Forms
+
+Template-driven forms use two-way data binding (`[(ngModel)]`) to update the data model in the component as changes are made in the template and vice versa. They are ideal for simple forms and use directives in the HTML template to manage form state and validation.
+
+## Core Directives
+
+Template-driven forms rely on the `FormsModule` which provides these key directives:
+
+- `NgModel`: Reconciles value changes in the form element with the data model (`[(ngModel)]`).
+- `NgForm`: Automatically creates a top-level `FormGroup` bound to the `<form>` tag.
+- `NgModelGroup`: Creates a nested `FormGroup` bound to a DOM element.
+
+## Setup
+
+First, import `FormsModule` into your component or module.
+
+```ts
+import {Component} from '@angular/core';
+import {FormsModule} from '@angular/forms';
+
+@Component({
+  selector: 'app-user-form',
+  imports: [FormsModule],
+  templateUrl: './user-form.component.html',
+})
+export class UserForm {
+  user = {name: '', role: 'Guest'};
+
+  onSubmit() {
+    console.log('Form submitted!', this.user);
+  }
+}
+```
+
+## Building the Form Template
+
+### Two-Way Binding with `[(ngModel)]`
+
+Use `[(ngModel)]` on input elements. **Every element using `[(ngModel)]` MUST have a `name` attribute.** Angular uses the `name` attribute to register the control with the parent `NgForm`.
+
+```html
+<form #userForm="ngForm" (ngSubmit)="onSubmit()">
+  <!-- Basic Input -->
+  <div>
+    <label for="name">Name:</label>
+    <input type="text" id="name" required [(ngModel)]="user.name" name="name" #nameCtrl="ngModel" />
+  </div>
+
+  <!-- Select Box -->
+  <div>
+    <label for="role">Role:</label>
+    <select id="role" [(ngModel)]="user.role" name="role">
+      <option value="Admin">Admin</option>
+      <option value="Guest">Guest</option>
+    </select>
+  </div>
+
+  <!-- Submit Button (disabled if form is invalid) -->
+  <button type="submit" [disabled]="!userForm.form.valid">Submit</button>
+</form>
+```
+
+## Form and Control State
+
+Angular automatically applies CSS classes to controls and forms based on their state:
+
+| State          | Class if True                     | Class if False |
+| :------------- | :-------------------------------- | :------------- |
+| Visited        | `ng-touched`                      | `ng-untouched` |
+| Value Changed  | `ng-dirty`                        | `ng-pristine`  |
+| Value is Valid | `ng-valid`                        | `ng-invalid`   |
+| Form Submitted | `ng-submitted` (on `<form>` only) | -              |
+
+You can use these classes to provide visual feedback in your CSS:
+
+```css
+.ng-valid[required],
+.ng-valid.required {
+  border-left: 5px solid #42a948; /* green */
+}
+.ng-invalid:not(form) {
+  border-left: 5px solid #a94442; /* red */
+}
+```
+
+## Validation and Error Messages
+
+To display error messages conditionally, export the `ngModel` directive to a template reference variable (e.g., `#nameCtrl="ngModel"`).
+
+```html
+<input type="text" id="name" required [(ngModel)]="user.name" name="name" #nameCtrl="ngModel" />
+
+<!-- Show error only if the control is invalid AND (touched OR dirty) -->
+@if (nameCtrl.invalid && (nameCtrl.dirty || nameCtrl.touched)) {
+<div class="alert alert-danger">
+  @if (nameCtrl.errors?.['required']) {
+  <div>Name is required.</div>
+  }
+</div>
+}
+```
+
+## Submitting the Form
+
+1. Use the `(ngSubmit)` event on the `<form>` element.
+2. Bind the submit button's disabled state to the overall form validity using the `NgForm` template reference variable (e.g., `[disabled]="!userForm.form.valid"`).
+
+## Resetting the Form
+
+To programmatically reset the form to its pristine state (clearing values and validation flags), use the `reset()` method on the `NgForm` instance.
+
+```html
+<button type="button" (click)="userForm.reset()">Reset</button>
+```

package/skills/angular-developer/references/testing-fundamentals.md

@@ -0,0 +1,63 @@
+# Testing Fundamentals
+
+This guide covers the fundamental principles and practices for writing unit tests in this repository, which uses Vitest as the test runner.
+
+## Core Philosophy: Zoneless & Async-First
+
+This project follows a modern, zoneless testing approach. State changes schedule updates asynchronously, and tests must account for this.
+
+**Do NOT** use `fixture.detectChanges()` to manually trigger updates.
+**ALWAYS** use the "Act, Wait, Assert" pattern:
+
+1.  **Act:** Update state or perform an action (e.g., set a component input, click a button).
+2.  **Wait:** Use `await fixture.whenStable()` to allow the framework to process the scheduled update and render the changes.
+3.  **Assert:** Verify the outcome.
+
+### Basic Test Structure Example
+
+```ts
+import {ComponentFixture, TestBed} from '@angular/core/testing';
+import {MyComponent} from './my.component';
+
+describe('MyComponent', () => {
+  let component: MyComponent;
+  let fixture: ComponentFixture<MyComponent>;
+  let h1: HTMLElement;
+
+  beforeEach(() => {
+    TestBed.configureTestingModule({});
+
+    // Create the component fixture
+    fixture = TestBed.createComponent(MyComponent);
+    component = fixture.componentInstance;
+    h1 = fixture.nativeElement.querySelector('h1');
+  });
+
+  it('should display the default title', async () => {
+    // ACT: (Implicit) Component is created with default state.
+    // WAIT for initial data binding.
+    await fixture.whenStable();
+    // ASSERT the initial state.
+    expect(h1.textContent).toContain('Default Title');
+  });
+
+  it('should display a different title after a change', async () => {
+    // ACT: Change the component's title property.
+    component.title.set('New Test Title');
+
+    // WAIT for the asynchronous update to complete.
+    await fixture.whenStable();
+
+    // ASSERT the DOM has been updated.
+    expect(h1.textContent).toContain('New Test Title');
+  });
+});
+```
+
+## TestBed and ComponentFixture
+
+- **`TestBed`**: The primary utility for creating a test-specific Angular module. Use `TestBed.configureTestingModule({...})` in your `beforeEach` to declare components, provide services, and set up imports needed for your test.
+- **`ComponentFixture`**: A handle on the created component instance and its environment.
+  - `fixture.componentInstance`: Access the component's class instance.
+  - `fixture.nativeElement`: Access the component's root DOM element.
+  - `fixture.debugElement`: An Angular-specific wrapper around the `nativeElement` that provides safer, platform-agnostic ways to query the DOM (e.g., `debugElement.query(By.css('p'))`).

package/skills/angular-developer/scripts/get-best-practices.mjs

@@ -0,0 +1,268 @@
+#!/usr/bin/env node
+
+import fs from 'node:fs/promises';
+import path from 'node:path';
+import process from 'node:process';
+import {createRequire} from 'node:module';
+import {pathToFileURL, fileURLToPath} from 'node:url';
+
+const MAX_BEST_PRACTICES_BYTES = 1_048_576;
+const BUILTIN_FALLBACK = `Bundled Angular best-practices guidance is unavailable in this workspace.\n\nTo get version-aware guidance, run this script from an Angular workspace with @angular/core installed and accessible via Node resolution.`;
+
+main().catch((error) => {
+  console.error(`Fatal: ${error instanceof Error ? error.message : String(error)}`);
+  process.exit(1);
+});
+
+async function main() {
+  const parsed = parseArguments(process.argv.slice(2));
+
+  if (parsed.help) {
+    printHelp();
+    process.exit(0);
+  }
+
+  if (parsed.errors.length > 0) {
+    console.error(parsed.errors.join('\n'));
+    console.error('Use --help for usage.');
+    process.exit(2);
+  }
+
+  const workspacePath = parsed.workspacePath || null;
+  const source = await resolveSource(workspacePath);
+
+  const payload = {
+    source: source.source,
+    content: source.content,
+  };
+
+  if (parsed.json) {
+    const json = parsed.pretty
+      ? JSON.stringify(payload, null, 2)
+      : JSON.stringify(payload);
+    process.stdout.write(json);
+    return;
+  }
+
+  console.log(`<!-- Source: ${payload.source} -->`);
+  console.log('');
+  console.log((payload.content || '').trimEnd());
+}
+
+function parseArguments(argv) {
+  const result = {
+    workspacePath: null,
+    json: false,
+    pretty: false,
+    help: false,
+    errors: [],
+  };
+
+  const positional = [];
+
+  for (let i = 0; i < argv.length; i++) {
+    const arg = argv[i];
+
+    if (!arg.startsWith('--')) {
+      positional.push(arg);
+      continue;
+    }
+
+    if (arg === '--json') {
+      result.json = true;
+      continue;
+    }
+
+    if (arg === '--pretty') {
+      result.pretty = true;
+      continue;
+    }
+
+    if (arg === '--help') {
+      result.help = true;
+      continue;
+    }
+
+    if (arg.startsWith('--')) {
+      result.errors.push(`Unknown option: ${arg}`);
+      continue;
+    }
+  }
+
+  if (positional.length > 0) {
+    result.workspacePath = positional[0];
+  }
+
+  return result;
+}
+
+async function resolveSource(workspacePath) {
+  if (workspacePath) {
+    try {
+      const workspaceRoot = await resolveWorkspaceFromInput(path.resolve(workspacePath));
+
+      if (!workspaceRoot) {
+        console.error(`Warning: could not locate an Angular workspace at "${workspacePath}". Falling back to bundled best-practices.md.`);
+      } else {
+        try {
+          const source = await loadWorkspaceBestPractices(workspaceRoot);
+          return source;
+        } catch (error) {
+          console.error(
+            `Warning: failed to read workspace-specific best practices from ${workspaceRoot}: ${error instanceof Error ? error.message : String(error)}. Falling back to bundled best-practices.md.`,
+          );
+        }
+      }
+    } catch (error) {
+      console.error(
+        `Warning: failed to locate workspace from ${workspacePath}: ${error instanceof Error ? error.message : String(error)}. Falling back to bundled best-practices.md.`,
+      );
+    }
+  } else {
+    const workspaceRoot = await findWorkspaceFromCwd(process.cwd());
+
+    if (workspaceRoot) {
+      try {
+        const source = await loadWorkspaceBestPractices(workspaceRoot);
+        return source;
+      } catch (error) {
+        console.error(
+          `Warning: failed to read workspace-specific best practices from ${workspaceRoot}: ${error instanceof Error ? error.message : String(error)}. Falling back to bundled best-practices.md.`,
+        );
+      }
+    }
+  }
+
+  const scriptDir = path.dirname(fileURLToPath(import.meta.url));
+  const fallbackPath = path.join(path.dirname(scriptDir), 'data', 'best-practices.md');
+  try {
+    const content = await fs.readFile(fallbackPath, 'utf8');
+    return {source: 'bundled best-practices fallback', content};
+  } catch (error) {
+    console.error(`Warning: bundled fallback not found at ${fallbackPath}: ${error instanceof Error ? error.message : String(error)}.`);
+    return {source: 'built-in fallback', content: BUILTIN_FALLBACK};
+  }
+}
+
+async function loadWorkspaceBestPractices(workspaceRoot) {
+  const packageJsonPath = path.join(workspaceRoot, 'package.json');
+  const probePath = path.join(workspaceRoot, '.pi-scripts-probe.mjs');
+  const requireBase = (await fileExists(packageJsonPath)) ? packageJsonPath : probePath;
+  const require = createRequire(pathToFileURL(requireBase).href);
+
+  const corePackageJsonPath = require.resolve('@angular/core/package.json');
+  const corePackageJson = JSON.parse(await fs.readFile(corePackageJsonPath, 'utf8'));
+
+  const bestPracticesMeta = corePackageJson.angular?.bestPractices;
+
+  if (!bestPracticesMeta || bestPracticesMeta.format !== 'markdown' || typeof bestPracticesMeta.path !== 'string') {
+    throw new Error('Unsupported Angular best-practices metadata format.');
+  }
+
+  const packageDirectory = path.dirname(corePackageJsonPath);
+  const bestPracticesPath = path.resolve(packageDirectory, bestPracticesMeta.path);
+
+  const rel = path.relative(packageDirectory, bestPracticesPath);
+  if (
+    rel.startsWith('..') ||
+    rel.startsWith(path.sep) ||
+    path.isAbsolute(bestPracticesMeta.path)
+  ) {
+    throw new Error('Best-practices path does not stay within @angular/core package directory.');
+  }
+
+  const stats = await fs.stat(bestPracticesPath);
+  if (!stats.isFile()) {
+    throw new Error(`Best-practices path is not a file: ${bestPracticesPath}`);
+  }
+
+  if (stats.size > MAX_BEST_PRACTICES_BYTES) {
+    throw new Error(`Best-practices file is too large (${stats.size} bytes).`);
+  }
+
+  const content = await fs.readFile(bestPracticesPath, 'utf8');
+  if (!content.trim()) {
+    throw new Error(`Best-practices file is empty: ${bestPracticesPath}`);
+  }
+
+  return {
+    source: `framework version ${corePackageJson.version}`,
+    content,
+  };
+}
+
+async function resolveWorkspaceFromInput(inputPath) {
+  const stats = await fs.stat(inputPath);
+
+  if (stats.isFile()) {
+    if (path.basename(inputPath) === 'angular.json') {
+      return path.dirname(inputPath);
+    }
+
+    throw new Error('Provided file path is not angular.json.');
+  }
+
+  if (stats.isDirectory()) {
+    const candidate = path.join(inputPath, 'angular.json');
+    if (await isFile(candidate)) {
+      return inputPath;
+    }
+    throw new Error('No angular.json found at the provided directory.');
+  }
+
+  throw new Error('Provided workspace path is neither a file nor directory.');
+}
+
+async function findWorkspaceFromCwd(startDir) {
+  let current = path.resolve(startDir);
+
+  for (;;) {
+    const candidate = path.join(current, 'angular.json');
+    if (await isFile(candidate)) {
+      return current;
+    }
+
+    const parent = path.dirname(current);
+    if (parent === current) {
+      return null;
+    }
+
+    current = parent;
+  }
+}
+
+async function isFile(candidate) {
+  try {
+    return (await fs.stat(candidate)).isFile();
+  } catch {
+    return false;
+  }
+}
+
+async function fileExists(candidate) {
+  try {
+    await fs.access(candidate);
+    return true;
+  } catch {
+    return false;
+  }
+}
+
+function printHelp() {
+  console.log(`
+Usage:
+  node scripts/get-best-practices.mjs [workspacePath] [--json] [--pretty] [--help]
+
+Arguments:
+  workspacePath   Optional Angular workspace directory or angular.json file path.
+
+Options:
+  --json        Output JSON only.
+  --pretty      Pretty-print JSON output.
+  --help        Show this help.
+
+Output:
+  Default: markdown with a source comment on top.
+  JSON:  { source, content }
+`);
+}