Skip to content

docs(skills): strengthen UI review rules#11377

Open
Alan-TheGentleman wants to merge 1 commit into
masterfrom
docs/ui-skill-review-rules
Open

docs(skills): strengthen UI review rules#11377
Alan-TheGentleman wants to merge 1 commit into
masterfrom
docs/ui-skill-review-rules

Conversation

@Alan-TheGentleman
Copy link
Copy Markdown
Contributor

@Alan-TheGentleman Alan-TheGentleman commented May 27, 2026

Context

Follow-up from the review on #11376 / PROWLER-1770.

That review surfaced three reusable process gaps in the UI skills: feature-local helper placement, stale UI error branches after contract changes, and missing negative tests for mapper/type-guard predicates.

Description

This PR strengthens the AI-agent skill guidance so future UI changes catch those issues earlier.

  • Clarifies that feature/domain ownership beats raw usage count for helper placement
  • Adds a required UI error-state replacement checklist
  • Adds TypeScript guidance requiring negative tests for type guards and mapper predicates

Steps to review

  1. Review skills/prowler-ui/SKILL.md for the feature-local helper and error-state checklist additions.
  2. Review skills/typescript/SKILL.md for type guard negative-test guidance.
  3. Confirm no generated AGENTS metadata changes are needed because skill metadata was not changed.

Checklist

Community Checklist

SDK/CLI

  • Are there new checks included in this PR? No
    • If so, do we need to update permissions for the provider? Not applicable.

UI

  • All issue/task requirements work as expected on the UI. Not applicable; no UI runtime changes.
  • If this PR adds or updates npm dependencies, include package-health evidence (maintenance, popularity, known vulnerabilities, license, release age) and explain why existing/native alternatives are insufficient. Not applicable; no dependency changes.
  • Screenshots/Video of the functionality flow (if applicable) - Mobile (X < 640px). Not applicable.
  • Screenshots/Video of the functionality flow (if applicable) - Table (640px > X < 1024px). Not applicable.
  • Screenshots/Video of the functionality flow (if applicable) - Desktop (X > 1024px). Not applicable.
  • Ensure new entries are added to CHANGELOG.md, if applicable. Not applicable; skill documentation only.

API

  • All issue/task requirements work as expected on the API. Not applicable.
  • Endpoint response output (if applicable). Not applicable.
  • EXPLAIN ANALYZE output for new/modified queries or indexes (if applicable). Not applicable.
  • Performance test results (if applicable). Not applicable.
  • Any other relevant evidence of the implementation (if applicable). Not applicable.
  • Verify if API specs need to be regenerated. Not applicable.
  • Check if version updates are required (e.g., specs, uv, etc.). Not applicable.
  • Ensure new entries are added to CHANGELOG.md, if applicable. Not applicable.

License

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

- Clarify feature-local helper placement
- Add UI error-state replacement checklist
- Require negative tests for mapper predicates
@Alan-TheGentleman Alan-TheGentleman added the no-changelog Skip including change in changelog/release notes label May 27, 2026
@github-actions
Copy link
Copy Markdown
Contributor

Conflict Markers Resolved

All conflict markers have been successfully resolved in this pull request.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

no-changelog Skip including change in changelog/release notes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant