CtrlK
BlogDocsLog inGet started
Tessl Logo
SkillPublic

review-fix-violations

Use when architecture violations were found and need automated fixing by priority.

Quality

81%

Does it follow best practices?

Run evals on this skill

Adds up to 20 points to the overall score

View guide
SecuritybySnyk

Passed

No findings from the security scan

SKILL.md
Quality
Evals
Security

Identify and fix docs/ARCHITECTURE.md violations in the specified module.

Module: $ARGUMENTS

Steps

  1. Run /review-check-architecture to find violations.

  2. For each violation found, fix in priority order:

    Critical (fix first):

    • try/catch in services -> remove, move error handling to component/interceptor
    • Transformation in services -> move to adapter
    • innerHTML without sanitization -> remove or use DomSanitizer
    • Hardcoded secrets -> move to environment files

    Important:

    • NgModule -> migrate to standalone (/migration-migrate-component)
    • @Input/@Output decorators -> input()/output() signals
    • Constructor DI -> inject()
    • BehaviorSubject in stores -> signals
    • any types -> type correctly
    • Missing OnPush -> add ChangeDetectionStrategy.OnPush

    Improvements:

    • Components > 200 lines -> decompose
    • Console.log -> remove
    • TODO/FIXME -> resolve or create issue
  3. Validate after each fix:

npx tsc --noEmit && ng build && ng test --watch=false
  1. Report what was fixed and what remains.
Repository

fernandezbaptiste/specialist-agent

Path

packs/angular/skills/review-fix-violations/SKILL.md

Last updated
First committed

Is this your skill?

If you maintain this skill, you can claim it as your own. Once claimed, you can manage eval scenarios, bundle related skills, attach documentation or rules, and ensure cross-agent compatibility.