@su-record/vibe 2.7.13 → 2.7.15
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
- package/.env.example +37 -37
- package/CLAUDE.md +134 -126
- package/LICENSE +21 -21
- package/README.md +449 -449
- package/agents/architect-low.md +41 -41
- package/agents/architect-medium.md +59 -59
- package/agents/architect.md +80 -80
- package/agents/build-error-resolver.md +115 -115
- package/agents/compounder.md +261 -261
- package/agents/diagrammer.md +178 -178
- package/agents/docs/api-documenter.md +99 -99
- package/agents/docs/changelog-writer.md +93 -93
- package/agents/e2e-tester.md +294 -294
- package/agents/explorer-low.md +42 -42
- package/agents/explorer-medium.md +59 -59
- package/agents/explorer.md +48 -48
- package/agents/implementer-low.md +43 -43
- package/agents/implementer-medium.md +52 -52
- package/agents/implementer.md +54 -54
- package/agents/junior-mentor.md +141 -141
- package/agents/planning/requirements-analyst.md +84 -84
- package/agents/planning/ux-advisor.md +83 -83
- package/agents/qa/acceptance-tester.md +86 -86
- package/agents/qa/edge-case-finder.md +93 -93
- package/agents/refactor-cleaner.md +143 -143
- package/agents/research/best-practices-agent.md +199 -199
- package/agents/research/codebase-patterns-agent.md +157 -157
- package/agents/research/framework-docs-agent.md +188 -188
- package/agents/research/security-advisory-agent.md +213 -213
- package/agents/review/architecture-reviewer.md +107 -107
- package/agents/review/complexity-reviewer.md +116 -116
- package/agents/review/data-integrity-reviewer.md +88 -88
- package/agents/review/git-history-reviewer.md +103 -103
- package/agents/review/performance-reviewer.md +86 -86
- package/agents/review/python-reviewer.md +150 -150
- package/agents/review/rails-reviewer.md +139 -139
- package/agents/review/react-reviewer.md +144 -144
- package/agents/review/security-reviewer.md +80 -80
- package/agents/review/simplicity-reviewer.md +140 -140
- package/agents/review/test-coverage-reviewer.md +116 -116
- package/agents/review/typescript-reviewer.md +127 -127
- package/agents/searcher.md +54 -54
- package/agents/simplifier.md +120 -120
- package/agents/tester.md +49 -49
- package/agents/ui/ui-a11y-auditor.md +93 -93
- package/agents/ui/ui-antipattern-detector.md +94 -94
- package/agents/ui/ui-dataviz-advisor.md +69 -69
- package/agents/ui/ui-design-system-gen.md +57 -57
- package/agents/ui/ui-industry-analyzer.md +49 -49
- package/agents/ui/ui-layout-architect.md +65 -65
- package/agents/ui/ui-stack-implementer.md +68 -68
- package/agents/ui/ux-compliance-reviewer.md +81 -81
- package/agents/ui-previewer.md +258 -260
- package/commands/vibe.analyze.md +11 -13
- package/commands/vibe.review.md +43 -1
- package/commands/vibe.run.md +2124 -2078
- package/commands/vibe.spec.md +9 -4
- package/commands/vibe.spec.review.md +569 -565
- package/commands/vibe.utils.md +413 -413
- package/commands/vibe.verify.md +33 -8
- package/dist/cli/collaborator.js +52 -52
- package/dist/cli/commands/evolution.js +12 -12
- package/dist/cli/commands/info.js +54 -54
- package/dist/cli/commands/init.js +5 -5
- package/dist/cli/commands/remove.js +14 -14
- package/dist/cli/commands/sentinel.js +27 -27
- package/dist/cli/commands/skills.js +5 -5
- package/dist/cli/commands/slack.js +10 -10
- package/dist/cli/commands/telegram.js +12 -12
- package/dist/cli/detect.js +32 -32
- package/dist/cli/index.js +51 -51
- package/dist/cli/llm/claude-commands.js +16 -16
- package/dist/cli/llm/config.js +19 -19
- package/dist/cli/llm/config.js.map +1 -1
- package/dist/cli/llm/gemini-commands.js +16 -16
- package/dist/cli/llm/gpt-commands.js +19 -19
- package/dist/cli/llm/help.js +21 -21
- package/dist/cli/postinstall/cursor-agents.js +32 -32
- package/dist/cli/postinstall/cursor-rules.js +83 -83
- package/dist/cli/postinstall/cursor-skills.js +743 -743
- package/dist/cli/setup/Provisioner.js +42 -42
- package/dist/cli/types.d.ts +0 -2
- package/dist/cli/types.d.ts.map +1 -1
- package/dist/infra/lib/DeepInit.js +24 -24
- package/dist/infra/lib/IterationTracker.js +11 -11
- package/dist/infra/lib/PythonParser.js +108 -108
- package/dist/infra/lib/ReviewRace.js +96 -96
- package/dist/infra/lib/SkillFrontmatter.js +28 -28
- package/dist/infra/lib/SkillQualityGate.js +9 -9
- package/dist/infra/lib/SkillRepository.js +159 -159
- package/dist/infra/lib/UltraQA.js +99 -99
- package/dist/infra/lib/autonomy/AuditStore.js +41 -41
- package/dist/infra/lib/autonomy/ConfirmationStore.js +30 -30
- package/dist/infra/lib/autonomy/EventOutbox.js +38 -38
- package/dist/infra/lib/autonomy/PolicyEngine.js +18 -18
- package/dist/infra/lib/autonomy/SecuritySentinel.js +1 -1
- package/dist/infra/lib/autonomy/SuggestionStore.js +33 -33
- package/dist/infra/lib/embedding/VectorStore.js +22 -22
- package/dist/infra/lib/evolution/AgentAnalyzer.js +10 -10
- package/dist/infra/lib/evolution/DescriptionOptimizer.js +21 -21
- package/dist/infra/lib/evolution/GenerationRegistry.js +36 -36
- package/dist/infra/lib/evolution/InsightStore.js +90 -90
- package/dist/infra/lib/evolution/RollbackManager.js +5 -5
- package/dist/infra/lib/evolution/SkillBenchmark.js +23 -23
- package/dist/infra/lib/evolution/SkillEvalRunner.js +50 -50
- package/dist/infra/lib/evolution/SkillGapDetector.js +10 -10
- package/dist/infra/lib/evolution/UsageTracker.js +28 -28
- package/dist/infra/lib/gemini/orchestration.js +5 -5
- package/dist/infra/lib/gpt/orchestration.js +4 -4
- package/dist/infra/lib/memory/KnowledgeGraph.js +4 -4
- package/dist/infra/lib/memory/MemorySearch.js +57 -57
- package/dist/infra/lib/memory/MemoryStorage.js +181 -181
- package/dist/infra/lib/memory/ObservationStore.js +28 -28
- package/dist/infra/lib/memory/ReflectionStore.js +30 -30
- package/dist/infra/lib/memory/SessionRAGRetriever.js +7 -7
- package/dist/infra/lib/memory/SessionRAGStore.js +225 -225
- package/dist/infra/lib/memory/SessionSummarizer.js +9 -9
- package/dist/infra/orchestrator/AgentManager.js +12 -12
- package/dist/infra/orchestrator/AgentRegistry.js +65 -65
- package/dist/infra/orchestrator/MultiLlmResearch.js +8 -8
- package/dist/infra/orchestrator/SwarmOrchestrator.test.js +16 -16
- package/dist/infra/orchestrator/parallelResearch.js +24 -24
- package/dist/tools/convention/analyzeComplexity.test.js +115 -115
- package/dist/tools/convention/validateCodeQuality.test.js +104 -104
- package/dist/tools/memory/createMemoryTimeline.js +10 -10
- package/dist/tools/memory/getMemoryGraph.js +12 -12
- package/dist/tools/memory/getSessionContext.js +9 -9
- package/dist/tools/memory/linkMemories.js +14 -14
- package/dist/tools/memory/listMemories.js +4 -4
- package/dist/tools/memory/recallMemory.js +4 -4
- package/dist/tools/memory/saveMemory.js +4 -4
- package/dist/tools/memory/searchMemoriesAdvanced.js +23 -23
- package/dist/tools/semantic/analyzeDependencyGraph.js +12 -12
- package/dist/tools/semantic/astGrep.test.js +6 -6
- package/dist/tools/spec/prdParser.test.js +171 -171
- package/dist/tools/spec/specGenerator.js +169 -169
- package/dist/tools/spec/traceabilityMatrix.js +64 -64
- package/dist/tools/spec/traceabilityMatrix.test.js +28 -28
- package/hooks/gemini-hooks.json +73 -73
- package/hooks/hooks.json +137 -137
- package/hooks/scripts/code-check.js +77 -70
- package/hooks/scripts/context-save.js +212 -212
- package/hooks/scripts/hud-status.js +291 -291
- package/hooks/scripts/keyword-detector.js +214 -214
- package/hooks/scripts/llm-orchestrate.js +475 -475
- package/hooks/scripts/post-edit.js +32 -32
- package/hooks/scripts/pre-tool-guard.js +125 -125
- package/hooks/scripts/prompt-dispatcher.js +185 -185
- package/hooks/scripts/sentinel-guard.js +104 -104
- package/hooks/scripts/session-start.js +106 -106
- package/hooks/scripts/stop-notify.js +209 -209
- package/hooks/scripts/utils.js +100 -100
- package/languages/csharp-unity.md +515 -515
- package/languages/gdscript-godot.md +470 -470
- package/languages/ruby-rails.md +489 -489
- package/languages/typescript-angular.md +433 -433
- package/languages/typescript-astro.md +416 -416
- package/languages/typescript-electron.md +406 -406
- package/languages/typescript-nestjs.md +524 -524
- package/languages/typescript-svelte.md +407 -407
- package/languages/typescript-tauri.md +365 -365
- package/package.json +121 -121
- package/skills/agents-md/SKILL.md +120 -120
- package/skills/arch-guard/SKILL.md +180 -180
- package/skills/brand-assets/SKILL.md +146 -146
- package/skills/capability-loop/SKILL.md +167 -167
- package/skills/characterization-test/SKILL.md +206 -206
- package/skills/commerce-patterns/SKILL.md +59 -59
- package/skills/commit-push-pr/SKILL.md +75 -75
- package/skills/context7-usage/SKILL.md +105 -105
- package/skills/core-capabilities/SKILL.md +48 -48
- package/skills/e2e-commerce/SKILL.md +57 -57
- package/skills/exec-plan/SKILL.md +147 -147
- package/skills/frontend-design/SKILL.md +73 -73
- package/skills/git-worktree/SKILL.md +72 -72
- package/skills/handoff/SKILL.md +109 -109
- package/skills/parallel-research/SKILL.md +87 -87
- package/skills/priority-todos/SKILL.md +63 -63
- package/skills/seo-checklist/SKILL.md +57 -57
- package/skills/techdebt/SKILL.md +122 -122
- package/skills/tool-fallback/SKILL.md +103 -103
- package/skills/typescript-advanced-types/SKILL.md +66 -65
- package/skills/ui-ux-pro-max/SKILL.md +206 -206
- package/skills/vercel-react-best-practices/SKILL.md +59 -59
- package/skills/video-production/SKILL.md +51 -51
- package/vibe/config.json +29 -29
- package/vibe/constitution.md +227 -227
- package/vibe/rules/principles/communication-guide.md +98 -98
- package/vibe/rules/principles/development-philosophy.md +52 -52
- package/vibe/rules/principles/quick-start.md +102 -102
- package/vibe/rules/quality/bdd-contract-testing.md +393 -393
- package/vibe/rules/quality/checklist.md +276 -276
- package/vibe/rules/quality/performance.md +236 -236
- package/vibe/rules/quality/testing-strategy.md +440 -440
- package/vibe/rules/standards/anti-patterns.md +541 -541
- package/vibe/rules/standards/code-structure.md +291 -291
- package/vibe/rules/standards/complexity-metrics.md +313 -313
- package/vibe/rules/standards/git-workflow.md +237 -237
- package/vibe/rules/standards/naming-conventions.md +198 -198
- package/vibe/rules/standards/security.md +305 -305
- package/vibe/rules/writing/document-style.md +74 -74
- package/vibe/setup.sh +31 -31
- package/vibe/templates/constitution-template.md +252 -252
- package/vibe/templates/contract-backend-template.md +526 -526
- package/vibe/templates/contract-frontend-template.md +599 -599
- package/vibe/templates/feature-template.md +96 -96
- package/vibe/templates/spec-template.md +221 -221
- package/vibe/ui-ux-data/charts.csv +26 -26
- package/vibe/ui-ux-data/colors.csv +97 -97
- package/vibe/ui-ux-data/icons.csv +101 -101
- package/vibe/ui-ux-data/landing.csv +31 -31
- package/vibe/ui-ux-data/products.csv +96 -96
- package/vibe/ui-ux-data/react-performance.csv +45 -45
- package/vibe/ui-ux-data/stacks/astro.csv +54 -54
- package/vibe/ui-ux-data/stacks/flutter.csv +53 -53
- package/vibe/ui-ux-data/stacks/html-tailwind.csv +56 -56
- package/vibe/ui-ux-data/stacks/jetpack-compose.csv +53 -53
- package/vibe/ui-ux-data/stacks/nextjs.csv +53 -53
- package/vibe/ui-ux-data/stacks/nuxt-ui.csv +51 -51
- package/vibe/ui-ux-data/stacks/nuxtjs.csv +59 -59
- package/vibe/ui-ux-data/stacks/react-native.csv +52 -52
- package/vibe/ui-ux-data/stacks/react.csv +54 -54
- package/vibe/ui-ux-data/stacks/shadcn.csv +61 -61
- package/vibe/ui-ux-data/stacks/svelte.csv +54 -54
- package/vibe/ui-ux-data/stacks/swiftui.csv +51 -51
- package/vibe/ui-ux-data/stacks/vue.csv +50 -50
- package/vibe/ui-ux-data/styles.csv +68 -68
- package/vibe/ui-ux-data/typography.csv +57 -57
- package/vibe/ui-ux-data/ui-reasoning.csv +101 -101
- package/vibe/ui-ux-data/ux-guidelines.csv +99 -99
- package/vibe/ui-ux-data/version.json +31 -31
- package/vibe/ui-ux-data/web-interface.csv +31 -31
|
@@ -1,433 +1,433 @@
|
|
|
1
|
-
# 🅰️ TypeScript + Angular Quality Rules
|
|
2
|
-
|
|
3
|
-
## Core Principles (inherited from core)
|
|
4
|
-
|
|
5
|
-
```markdown
|
|
6
|
-
✅ Single Responsibility (SRP)
|
|
7
|
-
✅ Don't Repeat Yourself (DRY)
|
|
8
|
-
✅ Reusability
|
|
9
|
-
✅ Low Complexity
|
|
10
|
-
✅ Functions ≤ 30 lines
|
|
11
|
-
✅ Nesting ≤ 3 levels
|
|
12
|
-
✅ Cyclomatic complexity ≤ 10
|
|
13
|
-
```
|
|
14
|
-
|
|
15
|
-
## Angular Architecture
|
|
16
|
-
|
|
17
|
-
```
|
|
18
|
-
┌─────────────────────────────────────────────┐
|
|
19
|
-
│ Component (UI + Logic) │
|
|
20
|
-
│ - Template, styles, event handling │
|
|
21
|
-
├─────────────────────────────────────────────┤
|
|
22
|
-
│ Service (Business Logic) │
|
|
23
|
-
│ - API calls, state management, utilities │
|
|
24
|
-
├─────────────────────────────────────────────┤
|
|
25
|
-
│ Module (Feature Organization) │
|
|
26
|
-
│ - Group components, services, routes │
|
|
27
|
-
└─────────────────────────────────────────────┘
|
|
28
|
-
```
|
|
29
|
-
|
|
30
|
-
## TypeScript/Angular Patterns
|
|
31
|
-
|
|
32
|
-
### 1. Standalone Component (Angular 17+)
|
|
33
|
-
|
|
34
|
-
```typescript
|
|
35
|
-
// ✅ Standalone Component (Recommended)
|
|
36
|
-
@Component({
|
|
37
|
-
selector: 'app-user-card',
|
|
38
|
-
standalone: true,
|
|
39
|
-
imports: [CommonModule, RouterLink],
|
|
40
|
-
template: `
|
|
41
|
-
<div class="card">
|
|
42
|
-
<h2>{{ user().name }}</h2>
|
|
43
|
-
<p>{{ user().email }}</p>
|
|
44
|
-
<button (click)="onEdit()">Edit</button>
|
|
45
|
-
</div>
|
|
46
|
-
`,
|
|
47
|
-
styles: [`
|
|
48
|
-
.card {
|
|
49
|
-
padding: 1rem;
|
|
50
|
-
border: 1px solid #ccc;
|
|
51
|
-
border-radius: 8px;
|
|
52
|
-
}
|
|
53
|
-
`]
|
|
54
|
-
})
|
|
55
|
-
export class UserCardComponent {
|
|
56
|
-
user = input.required<User>();
|
|
57
|
-
edit = output<User>();
|
|
58
|
-
|
|
59
|
-
onEdit(): void {
|
|
60
|
-
this.edit.emit(this.user());
|
|
61
|
-
}
|
|
62
|
-
}
|
|
63
|
-
|
|
64
|
-
// ❌ NgModule-based (Legacy)
|
|
65
|
-
@NgModule({
|
|
66
|
-
declarations: [UserCardComponent],
|
|
67
|
-
imports: [CommonModule],
|
|
68
|
-
exports: [UserCardComponent],
|
|
69
|
-
})
|
|
70
|
-
export class UserModule {}
|
|
71
|
-
```
|
|
72
|
-
|
|
73
|
-
### 2. Signal-based State Management (Angular 17+)
|
|
74
|
-
|
|
75
|
-
```typescript
|
|
76
|
-
// ✅ Using Signals
|
|
77
|
-
@Component({
|
|
78
|
-
selector: 'app-counter',
|
|
79
|
-
standalone: true,
|
|
80
|
-
template: `
|
|
81
|
-
<div>
|
|
82
|
-
<p>Count: {{ count() }}</p>
|
|
83
|
-
<p>Double: {{ doubleCount() }}</p>
|
|
84
|
-
<button (click)="increment()">+</button>
|
|
85
|
-
<button (click)="decrement()">-</button>
|
|
86
|
-
</div>
|
|
87
|
-
`
|
|
88
|
-
})
|
|
89
|
-
export class CounterComponent {
|
|
90
|
-
count = signal(0);
|
|
91
|
-
doubleCount = computed(() => this.count() * 2);
|
|
92
|
-
|
|
93
|
-
increment(): void {
|
|
94
|
-
this.count.update(c => c + 1);
|
|
95
|
-
}
|
|
96
|
-
|
|
97
|
-
decrement(): void {
|
|
98
|
-
this.count.update(c => c - 1);
|
|
99
|
-
}
|
|
100
|
-
}
|
|
101
|
-
|
|
102
|
-
// ✅ Service with Signals
|
|
103
|
-
@Injectable({ providedIn: 'root' })
|
|
104
|
-
export class UserStore {
|
|
105
|
-
private readonly _users = signal<User[]>([]);
|
|
106
|
-
private readonly _loading = signal(false);
|
|
107
|
-
private readonly _error = signal<string | null>(null);
|
|
108
|
-
|
|
109
|
-
readonly users = this._users.asReadonly();
|
|
110
|
-
readonly loading = this._loading.asReadonly();
|
|
111
|
-
readonly error = this._error.asReadonly();
|
|
112
|
-
|
|
113
|
-
readonly activeUsers = computed(() =>
|
|
114
|
-
this._users().filter(u => u.isActive)
|
|
115
|
-
);
|
|
116
|
-
|
|
117
|
-
async loadUsers(): Promise<void> {
|
|
118
|
-
this._loading.set(true);
|
|
119
|
-
this._error.set(null);
|
|
120
|
-
try {
|
|
121
|
-
const users = await this.http.get<User[]>('/api/users').toPromise();
|
|
122
|
-
this._users.set(users ?? []);
|
|
123
|
-
} catch (e) {
|
|
124
|
-
this._error.set('Failed to load users');
|
|
125
|
-
} finally {
|
|
126
|
-
this._loading.set(false);
|
|
127
|
-
}
|
|
128
|
-
}
|
|
129
|
-
|
|
130
|
-
addUser(user: User): void {
|
|
131
|
-
this._users.update(users => [...users, user]);
|
|
132
|
-
}
|
|
133
|
-
}
|
|
134
|
-
```
|
|
135
|
-
|
|
136
|
-
### 3. New Control Flow (Angular 17+)
|
|
137
|
-
|
|
138
|
-
```typescript
|
|
139
|
-
// ✅ New @if, @for, @switch
|
|
140
|
-
@Component({
|
|
141
|
-
selector: 'app-user-list',
|
|
142
|
-
standalone: true,
|
|
143
|
-
template: `
|
|
144
|
-
@if (loading()) {
|
|
145
|
-
<app-spinner />
|
|
146
|
-
} @else if (error()) {
|
|
147
|
-
<p class="error">{{ error() }}</p>
|
|
148
|
-
} @else {
|
|
149
|
-
<ul>
|
|
150
|
-
@for (user of users(); track user.id) {
|
|
151
|
-
<li>{{ user.name }}</li>
|
|
152
|
-
} @empty {
|
|
153
|
-
<li>No users found</li>
|
|
154
|
-
}
|
|
155
|
-
</ul>
|
|
156
|
-
}
|
|
157
|
-
|
|
158
|
-
@switch (status()) {
|
|
159
|
-
@case ('active') {
|
|
160
|
-
<span class="badge-active">Active</span>
|
|
161
|
-
}
|
|
162
|
-
@case ('inactive') {
|
|
163
|
-
<span class="badge-inactive">Inactive</span>
|
|
164
|
-
}
|
|
165
|
-
@default {
|
|
166
|
-
<span>Unknown</span>
|
|
167
|
-
}
|
|
168
|
-
}
|
|
169
|
-
`
|
|
170
|
-
})
|
|
171
|
-
export class UserListComponent {
|
|
172
|
-
users = input.required<User[]>();
|
|
173
|
-
loading = input(false);
|
|
174
|
-
error = input<string | null>(null);
|
|
175
|
-
status = input<'active' | 'inactive' | null>(null);
|
|
176
|
-
}
|
|
177
|
-
|
|
178
|
-
// ❌ Old *ngIf, *ngFor (Legacy)
|
|
179
|
-
// <div *ngIf="loading">...</div>
|
|
180
|
-
// <li *ngFor="let user of users">...</li>
|
|
181
|
-
```
|
|
182
|
-
|
|
183
|
-
### 4. HTTP Client + Error Handling
|
|
184
|
-
|
|
185
|
-
```typescript
|
|
186
|
-
// ✅ HTTP Service
|
|
187
|
-
@Injectable({ providedIn: 'root' })
|
|
188
|
-
export class UserService {
|
|
189
|
-
private readonly http = inject(HttpClient);
|
|
190
|
-
private readonly baseUrl = '/api/users';
|
|
191
|
-
|
|
192
|
-
getAll(): Observable<User[]> {
|
|
193
|
-
return this.http.get<User[]>(this.baseUrl).pipe(
|
|
194
|
-
catchError(this.handleError)
|
|
195
|
-
);
|
|
196
|
-
}
|
|
197
|
-
|
|
198
|
-
getById(id: string): Observable<User> {
|
|
199
|
-
return this.http.get<User>(`${this.baseUrl}/${id}`).pipe(
|
|
200
|
-
catchError(this.handleError)
|
|
201
|
-
);
|
|
202
|
-
}
|
|
203
|
-
|
|
204
|
-
create(user: CreateUserDto): Observable<User> {
|
|
205
|
-
return this.http.post<User>(this.baseUrl, user).pipe(
|
|
206
|
-
catchError(this.handleError)
|
|
207
|
-
);
|
|
208
|
-
}
|
|
209
|
-
|
|
210
|
-
update(id: string, user: UpdateUserDto): Observable<User> {
|
|
211
|
-
return this.http.patch<User>(`${this.baseUrl}/${id}`, user).pipe(
|
|
212
|
-
catchError(this.handleError)
|
|
213
|
-
);
|
|
214
|
-
}
|
|
215
|
-
|
|
216
|
-
delete(id: string): Observable<void> {
|
|
217
|
-
return this.http.delete<void>(`${this.baseUrl}/${id}`).pipe(
|
|
218
|
-
catchError(this.handleError)
|
|
219
|
-
);
|
|
220
|
-
}
|
|
221
|
-
|
|
222
|
-
private handleError(error: HttpErrorResponse): Observable<never> {
|
|
223
|
-
let message = 'An error occurred';
|
|
224
|
-
if (error.status === 404) {
|
|
225
|
-
message = 'Resource not found';
|
|
226
|
-
} else if (error.status === 401) {
|
|
227
|
-
message = 'Unauthorized';
|
|
228
|
-
} else if (error.error?.message) {
|
|
229
|
-
message = error.error.message;
|
|
230
|
-
}
|
|
231
|
-
return throwError(() => new Error(message));
|
|
232
|
-
}
|
|
233
|
-
}
|
|
234
|
-
```
|
|
235
|
-
|
|
236
|
-
### 5. Reactive Forms
|
|
237
|
-
|
|
238
|
-
```typescript
|
|
239
|
-
// ✅ Typed Reactive Forms
|
|
240
|
-
@Component({
|
|
241
|
-
selector: 'app-user-form',
|
|
242
|
-
standalone: true,
|
|
243
|
-
imports: [ReactiveFormsModule],
|
|
244
|
-
template: `
|
|
245
|
-
<form [formGroup]="form" (ngSubmit)="onSubmit()">
|
|
246
|
-
<input formControlName="name" placeholder="Name">
|
|
247
|
-
@if (form.controls.name.errors?.['required']) {
|
|
248
|
-
<span class="error">Name is required</span>
|
|
249
|
-
}
|
|
250
|
-
|
|
251
|
-
<input formControlName="email" type="email" placeholder="Email">
|
|
252
|
-
@if (form.controls.email.errors?.['email']) {
|
|
253
|
-
<span class="error">Invalid email</span>
|
|
254
|
-
}
|
|
255
|
-
|
|
256
|
-
<button type="submit" [disabled]="form.invalid || submitting()">
|
|
257
|
-
{{ submitting() ? 'Saving...' : 'Save' }}
|
|
258
|
-
</button>
|
|
259
|
-
</form>
|
|
260
|
-
`
|
|
261
|
-
})
|
|
262
|
-
export class UserFormComponent {
|
|
263
|
-
private readonly fb = inject(NonNullableFormBuilder);
|
|
264
|
-
private readonly userService = inject(UserService);
|
|
265
|
-
|
|
266
|
-
submitting = signal(false);
|
|
267
|
-
|
|
268
|
-
form = this.fb.group({
|
|
269
|
-
name: ['', [Validators.required, Validators.minLength(2)]],
|
|
270
|
-
email: ['', [Validators.required, Validators.email]],
|
|
271
|
-
age: [null as number | null, [Validators.min(0), Validators.max(150)]],
|
|
272
|
-
});
|
|
273
|
-
|
|
274
|
-
onSubmit(): void {
|
|
275
|
-
if (this.form.invalid) return;
|
|
276
|
-
|
|
277
|
-
this.submitting.set(true);
|
|
278
|
-
this.userService.create(this.form.getRawValue()).subscribe({
|
|
279
|
-
next: () => {
|
|
280
|
-
this.form.reset();
|
|
281
|
-
this.submitting.set(false);
|
|
282
|
-
},
|
|
283
|
-
error: () => this.submitting.set(false),
|
|
284
|
-
});
|
|
285
|
-
}
|
|
286
|
-
}
|
|
287
|
-
```
|
|
288
|
-
|
|
289
|
-
### 6. Route Configuration (Standalone)
|
|
290
|
-
|
|
291
|
-
```typescript
|
|
292
|
-
// ✅ app.routes.ts
|
|
293
|
-
export const routes: Routes = [
|
|
294
|
-
{ path: '', redirectTo: 'home', pathMatch: 'full' },
|
|
295
|
-
{
|
|
296
|
-
path: 'home',
|
|
297
|
-
loadComponent: () => import('./home/home.component').then(m => m.HomeComponent),
|
|
298
|
-
},
|
|
299
|
-
{
|
|
300
|
-
path: 'users',
|
|
301
|
-
loadChildren: () => import('./user/user.routes').then(m => m.userRoutes),
|
|
302
|
-
canActivate: [authGuard],
|
|
303
|
-
},
|
|
304
|
-
{ path: '**', redirectTo: 'home' },
|
|
305
|
-
];
|
|
306
|
-
|
|
307
|
-
// ✅ user.routes.ts (Lazy loaded)
|
|
308
|
-
export const userRoutes: Routes = [
|
|
309
|
-
{
|
|
310
|
-
path: '',
|
|
311
|
-
loadComponent: () => import('./user-list.component').then(m => m.UserListComponent),
|
|
312
|
-
},
|
|
313
|
-
{
|
|
314
|
-
path: ':id',
|
|
315
|
-
loadComponent: () => import('./user-detail.component').then(m => m.UserDetailComponent),
|
|
316
|
-
resolve: { user: userResolver },
|
|
317
|
-
},
|
|
318
|
-
];
|
|
319
|
-
|
|
320
|
-
// ✅ Functional Guard
|
|
321
|
-
export const authGuard: CanActivateFn = (route, state) => {
|
|
322
|
-
const authService = inject(AuthService);
|
|
323
|
-
const router = inject(Router);
|
|
324
|
-
|
|
325
|
-
if (authService.isAuthenticated()) {
|
|
326
|
-
return true;
|
|
327
|
-
}
|
|
328
|
-
|
|
329
|
-
return router.createUrlTree(['/login'], {
|
|
330
|
-
queryParams: { returnUrl: state.url },
|
|
331
|
-
});
|
|
332
|
-
};
|
|
333
|
-
|
|
334
|
-
// ✅ Functional Resolver
|
|
335
|
-
export const userResolver: ResolveFn<User> = (route) => {
|
|
336
|
-
const userService = inject(UserService);
|
|
337
|
-
const id = route.paramMap.get('id')!;
|
|
338
|
-
return userService.getById(id);
|
|
339
|
-
};
|
|
340
|
-
```
|
|
341
|
-
|
|
342
|
-
### 7. Dependency Injection (inject function)
|
|
343
|
-
|
|
344
|
-
```typescript
|
|
345
|
-
// ✅ Using inject() function (Recommended)
|
|
346
|
-
@Component({
|
|
347
|
-
selector: 'app-user-page',
|
|
348
|
-
standalone: true,
|
|
349
|
-
template: `...`
|
|
350
|
-
})
|
|
351
|
-
export class UserPageComponent {
|
|
352
|
-
private readonly userService = inject(UserService);
|
|
353
|
-
private readonly route = inject(ActivatedRoute);
|
|
354
|
-
private readonly router = inject(Router);
|
|
355
|
-
|
|
356
|
-
user = toSignal(
|
|
357
|
-
this.route.paramMap.pipe(
|
|
358
|
-
switchMap(params => this.userService.getById(params.get('id')!))
|
|
359
|
-
)
|
|
360
|
-
);
|
|
361
|
-
}
|
|
362
|
-
|
|
363
|
-
// ❌ Constructor injection (Legacy)
|
|
364
|
-
export class UserPageComponent {
|
|
365
|
-
constructor(
|
|
366
|
-
private userService: UserService,
|
|
367
|
-
private route: ActivatedRoute,
|
|
368
|
-
) {}
|
|
369
|
-
}
|
|
370
|
-
```
|
|
371
|
-
|
|
372
|
-
### 8. Error Boundaries + Loading
|
|
373
|
-
|
|
374
|
-
```typescript
|
|
375
|
-
// ✅ defer + loading/error handling
|
|
376
|
-
@Component({
|
|
377
|
-
selector: 'app-dashboard',
|
|
378
|
-
standalone: true,
|
|
379
|
-
template: `
|
|
380
|
-
@defer (on viewport) {
|
|
381
|
-
<app-heavy-chart [data]="chartData()" />
|
|
382
|
-
} @placeholder {
|
|
383
|
-
<div class="skeleton">Loading chart...</div>
|
|
384
|
-
} @loading (minimum 500ms) {
|
|
385
|
-
<app-spinner />
|
|
386
|
-
} @error {
|
|
387
|
-
<p>Failed to load chart</p>
|
|
388
|
-
}
|
|
389
|
-
`
|
|
390
|
-
})
|
|
391
|
-
export class DashboardComponent {
|
|
392
|
-
chartData = signal<ChartData | null>(null);
|
|
393
|
-
}
|
|
394
|
-
```
|
|
395
|
-
|
|
396
|
-
## Recommended Folder Structure
|
|
397
|
-
|
|
398
|
-
```
|
|
399
|
-
src/app/
|
|
400
|
-
├── app.component.ts
|
|
401
|
-
├── app.config.ts
|
|
402
|
-
├── app.routes.ts
|
|
403
|
-
├── core/ # Singleton services
|
|
404
|
-
│ ├── auth/
|
|
405
|
-
│ ├── http/
|
|
406
|
-
│ └── guards/
|
|
407
|
-
├── shared/ # Shared components
|
|
408
|
-
│ ├── components/
|
|
409
|
-
│ ├── directives/
|
|
410
|
-
│ └── pipes/
|
|
411
|
-
├── features/ # Feature modules
|
|
412
|
-
│ ├── user/
|
|
413
|
-
│ │ ├── user-list.component.ts
|
|
414
|
-
│ │ ├── user-detail.component.ts
|
|
415
|
-
│ │ ├── user.service.ts
|
|
416
|
-
│ │ └── user.routes.ts
|
|
417
|
-
│ └── product/
|
|
418
|
-
└── models/ # Types/Interfaces
|
|
419
|
-
└── user.model.ts
|
|
420
|
-
```
|
|
421
|
-
|
|
422
|
-
## Checklist
|
|
423
|
-
|
|
424
|
-
- [ ] Use Standalone Components
|
|
425
|
-
- [ ] Signal-based state management
|
|
426
|
-
- [ ] New Control Flow (@if, @for)
|
|
427
|
-
- [ ] DI with inject() function
|
|
428
|
-
- [ ] Typed Reactive Forms
|
|
429
|
-
- [ ] Lazy Loading Routes
|
|
430
|
-
- [ ] Functional Guard/Resolver
|
|
431
|
-
- [ ] OnPush Change Detection
|
|
432
|
-
- [ ] Use trackBy function (@for track)
|
|
433
|
-
- [ ] Proper error handling
|
|
1
|
+
# 🅰️ TypeScript + Angular Quality Rules
|
|
2
|
+
|
|
3
|
+
## Core Principles (inherited from core)
|
|
4
|
+
|
|
5
|
+
```markdown
|
|
6
|
+
✅ Single Responsibility (SRP)
|
|
7
|
+
✅ Don't Repeat Yourself (DRY)
|
|
8
|
+
✅ Reusability
|
|
9
|
+
✅ Low Complexity
|
|
10
|
+
✅ Functions ≤ 30 lines
|
|
11
|
+
✅ Nesting ≤ 3 levels
|
|
12
|
+
✅ Cyclomatic complexity ≤ 10
|
|
13
|
+
```
|
|
14
|
+
|
|
15
|
+
## Angular Architecture
|
|
16
|
+
|
|
17
|
+
```
|
|
18
|
+
┌─────────────────────────────────────────────┐
|
|
19
|
+
│ Component (UI + Logic) │
|
|
20
|
+
│ - Template, styles, event handling │
|
|
21
|
+
├─────────────────────────────────────────────┤
|
|
22
|
+
│ Service (Business Logic) │
|
|
23
|
+
│ - API calls, state management, utilities │
|
|
24
|
+
├─────────────────────────────────────────────┤
|
|
25
|
+
│ Module (Feature Organization) │
|
|
26
|
+
│ - Group components, services, routes │
|
|
27
|
+
└─────────────────────────────────────────────┘
|
|
28
|
+
```
|
|
29
|
+
|
|
30
|
+
## TypeScript/Angular Patterns
|
|
31
|
+
|
|
32
|
+
### 1. Standalone Component (Angular 17+)
|
|
33
|
+
|
|
34
|
+
```typescript
|
|
35
|
+
// ✅ Standalone Component (Recommended)
|
|
36
|
+
@Component({
|
|
37
|
+
selector: 'app-user-card',
|
|
38
|
+
standalone: true,
|
|
39
|
+
imports: [CommonModule, RouterLink],
|
|
40
|
+
template: `
|
|
41
|
+
<div class="card">
|
|
42
|
+
<h2>{{ user().name }}</h2>
|
|
43
|
+
<p>{{ user().email }}</p>
|
|
44
|
+
<button (click)="onEdit()">Edit</button>
|
|
45
|
+
</div>
|
|
46
|
+
`,
|
|
47
|
+
styles: [`
|
|
48
|
+
.card {
|
|
49
|
+
padding: 1rem;
|
|
50
|
+
border: 1px solid #ccc;
|
|
51
|
+
border-radius: 8px;
|
|
52
|
+
}
|
|
53
|
+
`]
|
|
54
|
+
})
|
|
55
|
+
export class UserCardComponent {
|
|
56
|
+
user = input.required<User>();
|
|
57
|
+
edit = output<User>();
|
|
58
|
+
|
|
59
|
+
onEdit(): void {
|
|
60
|
+
this.edit.emit(this.user());
|
|
61
|
+
}
|
|
62
|
+
}
|
|
63
|
+
|
|
64
|
+
// ❌ NgModule-based (Legacy)
|
|
65
|
+
@NgModule({
|
|
66
|
+
declarations: [UserCardComponent],
|
|
67
|
+
imports: [CommonModule],
|
|
68
|
+
exports: [UserCardComponent],
|
|
69
|
+
})
|
|
70
|
+
export class UserModule {}
|
|
71
|
+
```
|
|
72
|
+
|
|
73
|
+
### 2. Signal-based State Management (Angular 17+)
|
|
74
|
+
|
|
75
|
+
```typescript
|
|
76
|
+
// ✅ Using Signals
|
|
77
|
+
@Component({
|
|
78
|
+
selector: 'app-counter',
|
|
79
|
+
standalone: true,
|
|
80
|
+
template: `
|
|
81
|
+
<div>
|
|
82
|
+
<p>Count: {{ count() }}</p>
|
|
83
|
+
<p>Double: {{ doubleCount() }}</p>
|
|
84
|
+
<button (click)="increment()">+</button>
|
|
85
|
+
<button (click)="decrement()">-</button>
|
|
86
|
+
</div>
|
|
87
|
+
`
|
|
88
|
+
})
|
|
89
|
+
export class CounterComponent {
|
|
90
|
+
count = signal(0);
|
|
91
|
+
doubleCount = computed(() => this.count() * 2);
|
|
92
|
+
|
|
93
|
+
increment(): void {
|
|
94
|
+
this.count.update(c => c + 1);
|
|
95
|
+
}
|
|
96
|
+
|
|
97
|
+
decrement(): void {
|
|
98
|
+
this.count.update(c => c - 1);
|
|
99
|
+
}
|
|
100
|
+
}
|
|
101
|
+
|
|
102
|
+
// ✅ Service with Signals
|
|
103
|
+
@Injectable({ providedIn: 'root' })
|
|
104
|
+
export class UserStore {
|
|
105
|
+
private readonly _users = signal<User[]>([]);
|
|
106
|
+
private readonly _loading = signal(false);
|
|
107
|
+
private readonly _error = signal<string | null>(null);
|
|
108
|
+
|
|
109
|
+
readonly users = this._users.asReadonly();
|
|
110
|
+
readonly loading = this._loading.asReadonly();
|
|
111
|
+
readonly error = this._error.asReadonly();
|
|
112
|
+
|
|
113
|
+
readonly activeUsers = computed(() =>
|
|
114
|
+
this._users().filter(u => u.isActive)
|
|
115
|
+
);
|
|
116
|
+
|
|
117
|
+
async loadUsers(): Promise<void> {
|
|
118
|
+
this._loading.set(true);
|
|
119
|
+
this._error.set(null);
|
|
120
|
+
try {
|
|
121
|
+
const users = await this.http.get<User[]>('/api/users').toPromise();
|
|
122
|
+
this._users.set(users ?? []);
|
|
123
|
+
} catch (e) {
|
|
124
|
+
this._error.set('Failed to load users');
|
|
125
|
+
} finally {
|
|
126
|
+
this._loading.set(false);
|
|
127
|
+
}
|
|
128
|
+
}
|
|
129
|
+
|
|
130
|
+
addUser(user: User): void {
|
|
131
|
+
this._users.update(users => [...users, user]);
|
|
132
|
+
}
|
|
133
|
+
}
|
|
134
|
+
```
|
|
135
|
+
|
|
136
|
+
### 3. New Control Flow (Angular 17+)
|
|
137
|
+
|
|
138
|
+
```typescript
|
|
139
|
+
// ✅ New @if, @for, @switch
|
|
140
|
+
@Component({
|
|
141
|
+
selector: 'app-user-list',
|
|
142
|
+
standalone: true,
|
|
143
|
+
template: `
|
|
144
|
+
@if (loading()) {
|
|
145
|
+
<app-spinner />
|
|
146
|
+
} @else if (error()) {
|
|
147
|
+
<p class="error">{{ error() }}</p>
|
|
148
|
+
} @else {
|
|
149
|
+
<ul>
|
|
150
|
+
@for (user of users(); track user.id) {
|
|
151
|
+
<li>{{ user.name }}</li>
|
|
152
|
+
} @empty {
|
|
153
|
+
<li>No users found</li>
|
|
154
|
+
}
|
|
155
|
+
</ul>
|
|
156
|
+
}
|
|
157
|
+
|
|
158
|
+
@switch (status()) {
|
|
159
|
+
@case ('active') {
|
|
160
|
+
<span class="badge-active">Active</span>
|
|
161
|
+
}
|
|
162
|
+
@case ('inactive') {
|
|
163
|
+
<span class="badge-inactive">Inactive</span>
|
|
164
|
+
}
|
|
165
|
+
@default {
|
|
166
|
+
<span>Unknown</span>
|
|
167
|
+
}
|
|
168
|
+
}
|
|
169
|
+
`
|
|
170
|
+
})
|
|
171
|
+
export class UserListComponent {
|
|
172
|
+
users = input.required<User[]>();
|
|
173
|
+
loading = input(false);
|
|
174
|
+
error = input<string | null>(null);
|
|
175
|
+
status = input<'active' | 'inactive' | null>(null);
|
|
176
|
+
}
|
|
177
|
+
|
|
178
|
+
// ❌ Old *ngIf, *ngFor (Legacy)
|
|
179
|
+
// <div *ngIf="loading">...</div>
|
|
180
|
+
// <li *ngFor="let user of users">...</li>
|
|
181
|
+
```
|
|
182
|
+
|
|
183
|
+
### 4. HTTP Client + Error Handling
|
|
184
|
+
|
|
185
|
+
```typescript
|
|
186
|
+
// ✅ HTTP Service
|
|
187
|
+
@Injectable({ providedIn: 'root' })
|
|
188
|
+
export class UserService {
|
|
189
|
+
private readonly http = inject(HttpClient);
|
|
190
|
+
private readonly baseUrl = '/api/users';
|
|
191
|
+
|
|
192
|
+
getAll(): Observable<User[]> {
|
|
193
|
+
return this.http.get<User[]>(this.baseUrl).pipe(
|
|
194
|
+
catchError(this.handleError)
|
|
195
|
+
);
|
|
196
|
+
}
|
|
197
|
+
|
|
198
|
+
getById(id: string): Observable<User> {
|
|
199
|
+
return this.http.get<User>(`${this.baseUrl}/${id}`).pipe(
|
|
200
|
+
catchError(this.handleError)
|
|
201
|
+
);
|
|
202
|
+
}
|
|
203
|
+
|
|
204
|
+
create(user: CreateUserDto): Observable<User> {
|
|
205
|
+
return this.http.post<User>(this.baseUrl, user).pipe(
|
|
206
|
+
catchError(this.handleError)
|
|
207
|
+
);
|
|
208
|
+
}
|
|
209
|
+
|
|
210
|
+
update(id: string, user: UpdateUserDto): Observable<User> {
|
|
211
|
+
return this.http.patch<User>(`${this.baseUrl}/${id}`, user).pipe(
|
|
212
|
+
catchError(this.handleError)
|
|
213
|
+
);
|
|
214
|
+
}
|
|
215
|
+
|
|
216
|
+
delete(id: string): Observable<void> {
|
|
217
|
+
return this.http.delete<void>(`${this.baseUrl}/${id}`).pipe(
|
|
218
|
+
catchError(this.handleError)
|
|
219
|
+
);
|
|
220
|
+
}
|
|
221
|
+
|
|
222
|
+
private handleError(error: HttpErrorResponse): Observable<never> {
|
|
223
|
+
let message = 'An error occurred';
|
|
224
|
+
if (error.status === 404) {
|
|
225
|
+
message = 'Resource not found';
|
|
226
|
+
} else if (error.status === 401) {
|
|
227
|
+
message = 'Unauthorized';
|
|
228
|
+
} else if (error.error?.message) {
|
|
229
|
+
message = error.error.message;
|
|
230
|
+
}
|
|
231
|
+
return throwError(() => new Error(message));
|
|
232
|
+
}
|
|
233
|
+
}
|
|
234
|
+
```
|
|
235
|
+
|
|
236
|
+
### 5. Reactive Forms
|
|
237
|
+
|
|
238
|
+
```typescript
|
|
239
|
+
// ✅ Typed Reactive Forms
|
|
240
|
+
@Component({
|
|
241
|
+
selector: 'app-user-form',
|
|
242
|
+
standalone: true,
|
|
243
|
+
imports: [ReactiveFormsModule],
|
|
244
|
+
template: `
|
|
245
|
+
<form [formGroup]="form" (ngSubmit)="onSubmit()">
|
|
246
|
+
<input formControlName="name" placeholder="Name">
|
|
247
|
+
@if (form.controls.name.errors?.['required']) {
|
|
248
|
+
<span class="error">Name is required</span>
|
|
249
|
+
}
|
|
250
|
+
|
|
251
|
+
<input formControlName="email" type="email" placeholder="Email">
|
|
252
|
+
@if (form.controls.email.errors?.['email']) {
|
|
253
|
+
<span class="error">Invalid email</span>
|
|
254
|
+
}
|
|
255
|
+
|
|
256
|
+
<button type="submit" [disabled]="form.invalid || submitting()">
|
|
257
|
+
{{ submitting() ? 'Saving...' : 'Save' }}
|
|
258
|
+
</button>
|
|
259
|
+
</form>
|
|
260
|
+
`
|
|
261
|
+
})
|
|
262
|
+
export class UserFormComponent {
|
|
263
|
+
private readonly fb = inject(NonNullableFormBuilder);
|
|
264
|
+
private readonly userService = inject(UserService);
|
|
265
|
+
|
|
266
|
+
submitting = signal(false);
|
|
267
|
+
|
|
268
|
+
form = this.fb.group({
|
|
269
|
+
name: ['', [Validators.required, Validators.minLength(2)]],
|
|
270
|
+
email: ['', [Validators.required, Validators.email]],
|
|
271
|
+
age: [null as number | null, [Validators.min(0), Validators.max(150)]],
|
|
272
|
+
});
|
|
273
|
+
|
|
274
|
+
onSubmit(): void {
|
|
275
|
+
if (this.form.invalid) return;
|
|
276
|
+
|
|
277
|
+
this.submitting.set(true);
|
|
278
|
+
this.userService.create(this.form.getRawValue()).subscribe({
|
|
279
|
+
next: () => {
|
|
280
|
+
this.form.reset();
|
|
281
|
+
this.submitting.set(false);
|
|
282
|
+
},
|
|
283
|
+
error: () => this.submitting.set(false),
|
|
284
|
+
});
|
|
285
|
+
}
|
|
286
|
+
}
|
|
287
|
+
```
|
|
288
|
+
|
|
289
|
+
### 6. Route Configuration (Standalone)
|
|
290
|
+
|
|
291
|
+
```typescript
|
|
292
|
+
// ✅ app.routes.ts
|
|
293
|
+
export const routes: Routes = [
|
|
294
|
+
{ path: '', redirectTo: 'home', pathMatch: 'full' },
|
|
295
|
+
{
|
|
296
|
+
path: 'home',
|
|
297
|
+
loadComponent: () => import('./home/home.component').then(m => m.HomeComponent),
|
|
298
|
+
},
|
|
299
|
+
{
|
|
300
|
+
path: 'users',
|
|
301
|
+
loadChildren: () => import('./user/user.routes').then(m => m.userRoutes),
|
|
302
|
+
canActivate: [authGuard],
|
|
303
|
+
},
|
|
304
|
+
{ path: '**', redirectTo: 'home' },
|
|
305
|
+
];
|
|
306
|
+
|
|
307
|
+
// ✅ user.routes.ts (Lazy loaded)
|
|
308
|
+
export const userRoutes: Routes = [
|
|
309
|
+
{
|
|
310
|
+
path: '',
|
|
311
|
+
loadComponent: () => import('./user-list.component').then(m => m.UserListComponent),
|
|
312
|
+
},
|
|
313
|
+
{
|
|
314
|
+
path: ':id',
|
|
315
|
+
loadComponent: () => import('./user-detail.component').then(m => m.UserDetailComponent),
|
|
316
|
+
resolve: { user: userResolver },
|
|
317
|
+
},
|
|
318
|
+
];
|
|
319
|
+
|
|
320
|
+
// ✅ Functional Guard
|
|
321
|
+
export const authGuard: CanActivateFn = (route, state) => {
|
|
322
|
+
const authService = inject(AuthService);
|
|
323
|
+
const router = inject(Router);
|
|
324
|
+
|
|
325
|
+
if (authService.isAuthenticated()) {
|
|
326
|
+
return true;
|
|
327
|
+
}
|
|
328
|
+
|
|
329
|
+
return router.createUrlTree(['/login'], {
|
|
330
|
+
queryParams: { returnUrl: state.url },
|
|
331
|
+
});
|
|
332
|
+
};
|
|
333
|
+
|
|
334
|
+
// ✅ Functional Resolver
|
|
335
|
+
export const userResolver: ResolveFn<User> = (route) => {
|
|
336
|
+
const userService = inject(UserService);
|
|
337
|
+
const id = route.paramMap.get('id')!;
|
|
338
|
+
return userService.getById(id);
|
|
339
|
+
};
|
|
340
|
+
```
|
|
341
|
+
|
|
342
|
+
### 7. Dependency Injection (inject function)
|
|
343
|
+
|
|
344
|
+
```typescript
|
|
345
|
+
// ✅ Using inject() function (Recommended)
|
|
346
|
+
@Component({
|
|
347
|
+
selector: 'app-user-page',
|
|
348
|
+
standalone: true,
|
|
349
|
+
template: `...`
|
|
350
|
+
})
|
|
351
|
+
export class UserPageComponent {
|
|
352
|
+
private readonly userService = inject(UserService);
|
|
353
|
+
private readonly route = inject(ActivatedRoute);
|
|
354
|
+
private readonly router = inject(Router);
|
|
355
|
+
|
|
356
|
+
user = toSignal(
|
|
357
|
+
this.route.paramMap.pipe(
|
|
358
|
+
switchMap(params => this.userService.getById(params.get('id')!))
|
|
359
|
+
)
|
|
360
|
+
);
|
|
361
|
+
}
|
|
362
|
+
|
|
363
|
+
// ❌ Constructor injection (Legacy)
|
|
364
|
+
export class UserPageComponent {
|
|
365
|
+
constructor(
|
|
366
|
+
private userService: UserService,
|
|
367
|
+
private route: ActivatedRoute,
|
|
368
|
+
) {}
|
|
369
|
+
}
|
|
370
|
+
```
|
|
371
|
+
|
|
372
|
+
### 8. Error Boundaries + Loading
|
|
373
|
+
|
|
374
|
+
```typescript
|
|
375
|
+
// ✅ defer + loading/error handling
|
|
376
|
+
@Component({
|
|
377
|
+
selector: 'app-dashboard',
|
|
378
|
+
standalone: true,
|
|
379
|
+
template: `
|
|
380
|
+
@defer (on viewport) {
|
|
381
|
+
<app-heavy-chart [data]="chartData()" />
|
|
382
|
+
} @placeholder {
|
|
383
|
+
<div class="skeleton">Loading chart...</div>
|
|
384
|
+
} @loading (minimum 500ms) {
|
|
385
|
+
<app-spinner />
|
|
386
|
+
} @error {
|
|
387
|
+
<p>Failed to load chart</p>
|
|
388
|
+
}
|
|
389
|
+
`
|
|
390
|
+
})
|
|
391
|
+
export class DashboardComponent {
|
|
392
|
+
chartData = signal<ChartData | null>(null);
|
|
393
|
+
}
|
|
394
|
+
```
|
|
395
|
+
|
|
396
|
+
## Recommended Folder Structure
|
|
397
|
+
|
|
398
|
+
```
|
|
399
|
+
src/app/
|
|
400
|
+
├── app.component.ts
|
|
401
|
+
├── app.config.ts
|
|
402
|
+
├── app.routes.ts
|
|
403
|
+
├── core/ # Singleton services
|
|
404
|
+
│ ├── auth/
|
|
405
|
+
│ ├── http/
|
|
406
|
+
│ └── guards/
|
|
407
|
+
├── shared/ # Shared components
|
|
408
|
+
│ ├── components/
|
|
409
|
+
│ ├── directives/
|
|
410
|
+
│ └── pipes/
|
|
411
|
+
├── features/ # Feature modules
|
|
412
|
+
│ ├── user/
|
|
413
|
+
│ │ ├── user-list.component.ts
|
|
414
|
+
│ │ ├── user-detail.component.ts
|
|
415
|
+
│ │ ├── user.service.ts
|
|
416
|
+
│ │ └── user.routes.ts
|
|
417
|
+
│ └── product/
|
|
418
|
+
└── models/ # Types/Interfaces
|
|
419
|
+
└── user.model.ts
|
|
420
|
+
```
|
|
421
|
+
|
|
422
|
+
## Checklist
|
|
423
|
+
|
|
424
|
+
- [ ] Use Standalone Components
|
|
425
|
+
- [ ] Signal-based state management
|
|
426
|
+
- [ ] New Control Flow (@if, @for)
|
|
427
|
+
- [ ] DI with inject() function
|
|
428
|
+
- [ ] Typed Reactive Forms
|
|
429
|
+
- [ ] Lazy Loading Routes
|
|
430
|
+
- [ ] Functional Guard/Resolver
|
|
431
|
+
- [ ] OnPush Change Detection
|
|
432
|
+
- [ ] Use trackBy function (@for track)
|
|
433
|
+
- [ ] Proper error handling
|