feat: improve workflow clarity and troubleshooting across 7 skills by popey · Pull Request #5 · analogjs/angular-skills
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 🙏