refactor(skills): reorganize skills under .agents

[kdinev]

Description

Provide a clear and concise summary of the changes in this PR and the motivation behind them

Type of Change

• Documentation update
• Refactoring (code improvements without functional changes)

Related Issues

Closes #

Testing

Checklist

• I have updated documentation if needed

[Copilot]

Copilot review overview

🟡 Changes recommended

Resolve the two moderate issues before approval.

Get a fresh assessment by requesting another Copilot review.

Review effort: Lite
Findings: 1 · 1

Open (2)

• Add required license field to skill frontmatter · New
• Update manifest comment to reference .agents/skills/ · New

What changed in this PR

This PR reorganizes contributor skills under .agents, updates documentation links, and adds ACS configuration and context.

Changes:

• Updates skill and guideline references.
• Adds skill-authoring guidance, manifest, permissions, and project context.
• Modifies the pre-commit hook.

File	Summary
skills/​README.md	Updated skill references.
.husky/​pre-commit	Comments out lint-staged; moderate issue (4 votes).
.github/​copilot-instructions.md	Updated skills location guidance.
.github/​CODING_GUIDELINES.md	Updated skills documentation link.
.agents/​skills/​update-component-styles/​SKILL.md	Relocated contributor workflow.
.agents/​skills/​skill-authoring/​SKILL.md	Added authoring guidance; missing required license field, moderate issue (4 votes).
.agents/​skills/​review-component-pr/​SKILL.md	Relocated contributor workflow.
.agents/​skills/​README.md	Documented skills; contains user-invokable typo, nit (3 votes).
.agents/​skills/​create-new-component/​SKILL.md	Relocated contributor workflow.
.agents/​skills/​add-component-property/​SKILL.md	Relocated contributor workflow.
.agents/​permissions/​policy.yaml	Added agent permission policy.
.agents/​main.yaml	Added ACS manifest; misleading skills-layer comment, nit (2 votes).
.agents/​context/​project.md	Added project context and workflow guidance.

---

💡 Configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

[Copilot]

Copilot review overview

🟡 Changes recommended

Address the critical nested .envrc permission gap and the skill metadata and consistency issues.

Get a fresh assessment by requesting another Copilot review.

Review effort: Lite
Findings: 1

Open (1)

• Nested .envrc files remain readable · New

Resolved since last review (2)

• Add required license field to skill frontmatter
• Update manifest comment to reference .agents/skills/

Previously missed (1)

In code that hasn't changed since last review

Existing skills do not follow the new frontmatter rules

.agents/​skills/​README.md:36

These new rules are immediately inconsistent with the existing contributor skills: create-new-component, add-component-property, update-component-styles, and review-component-pr still have only name and description frontmatter, with no license or WHEN TO USE:/WHEN NOT TO USE: markers. Either migrate those skills in this PR or state that the rules apply only to newly created skills.

This issue also appears on line 35 of the same file.

[Copilot]

Copilot review overview

🔵 Needs a closer look

The authoring skill omits its required license field, and the README guidance has consistency issues.

Review effort: Lite
Findings: None

Resolved since last review (1)

• Nested .envrc files remain readable

[Copilot]

Copilot review overview

🟡 Changes recommended

A critical permissions issue and several documentation inconsistencies remain unresolved.

Get a fresh assessment by requesting another Copilot review.

Review effort: Lite
Findings: 1 · 1

Open (2)

• Root-level sensitive files remain readable · New
• Published README contains dead links to uncopied files · New

[Copilot]

Copilot review overview

🟡 Changes recommended

Correct the documented user-invocable spelling and make the skill description comply with the new authoring rule.

Get a fresh assessment by requesting another Copilot review.

Review effort: Lite
Findings: 1

Open (1)

• Make skill description comply with third-person authoring rule · New

Resolved since last review (2)

• Root-level sensitive files remain readable
• Published README contains dead links to uncopied files

[Copilot]

Copilot review overview

🟡 Changes recommended

The permissions policy leaves protected files writable, and the skill documentation has consistency issues.

Get a fresh assessment by requesting another Copilot review.

Review effort: Lite
Findings: 1

Open (1)

• Broad write access allows overwriting protected secret and config files · New

Resolved since last review (1)

• Make skill description comply with third-person authoring rule

Previously missed (1)

In code that hasn't changed since last review

Incorrect user-invokable frontmatter key spelling

.agents/​skills/​README.md:35

This documents the frontmatter key as user-invokable, but the skill format and the new skill-authoring skill use user-invocable. Following this instruction would silently produce an unrecognized key, so change it to the valid spelling.