feat: improve workflow clarity and troubleshooting across 7 skills by popey · Pull Request #5 · analogjs/angular-skills

@popey

Hullo @analogjs 👋

I ran your skills through `tessl skill review` at work and found some targeted improvements. Here's the before/after:

| Skill | Before | After | Change |
|-------|--------|-------|--------|
| angular-di | 94% | 100% | +6% |
| angular-forms | 94% | 100% | +6% |
| angular-http | 94% | 100% | +6% |
| angular-routing | 94% | 100% | +6% |
| angular-ssr | 94% | 100% | +6% |
| angular-testing | 94% | 100% | +6% |
| angular-tooling | 94% | 100% | +6% |

The remaining three skills (angular-component, angular-directives, angular-signals) already scored 100% — no changes needed.

<details>
<summary>Changes made</summary>

- **angular-di**: Added a provider scope decision matrix (root vs component vs route), a "Validating DI Configuration" section with console verification patterns, and a "Troubleshooting Common DI Errors" section covering NullInjectorError, inject() context errors, and circular dependencies.
- **angular-forms**: Added a "Recommended Workflow" section with a 5-step form-building sequence, expanded the Form Submission example with error handling (try/catch + serverError signal), and added a "Server-Side Validation Errors" subsection showing how to surface API field errors via reactive validators.
- **angular-http**: Added a "Choosing the Right Approach" decision matrix (httpResource vs resource vs HttpClient), and added "Error recovery validation" checklists under both httpResource and HttpClient error-handling sections.
- **angular-routing**: Added a "Setup Workflow" section with a numbered step-by-step sequence for adding routing, and added "Verify:" hints after Auth Guard and Role Guard examples.
- **angular-ssr**: Added a "Troubleshooting" section covering hydration mismatches (NG0500/NG0501 diagnosis, common causes table, ngSkipHydration, verify-fix-retry workflow) and build failures (browser globals, prerender route errors).
- **angular-testing**: Added a "Setup Verification & Common Issues" subsection with a troubleshooting table for the six most common test failure modes (missing detectChanges, stale signals, missing providers, async timing, HTTP mocking).
- **angular-tooling**: Added "Verify Build Success" and "Verify Update Success" subsections with post-operation verification commands and common failure recovery guidance.

</details>

Honest disclosure — I work at @tesslio where we build tooling around skills like these. Not a pitch - just saw room for improvement and wanted to contribute.

If you want to run evals yourself, just `npm install @tessl/cli` then run `tessl skill review path/to/your/SKILL.md`, and click [here](https://tessl.io/registry/skills/submit) to find out more.

Thanks in advance 🙏