feat: add technical-debt-identifier skill and microservices-architect…

[f-lrz]

Description

This PR introduces a new Skill and a new Agent to help teams maintain code quality and scale their architecture, fulfilling the requirements for the #awesome-copilot challenge.

🧰 Skill: technical-debt-identifier

Analyzes legacy code and provides a structured Technical Debt Report, identifying code smells, architecture violations, and generating a prioritized refactoring table.

• Extra Asset: Included legacy-auth.js in the examples folder to demonstrate how the skill catches hardcoded credentials and dead code.

🎭 Agent: microservices-architect

A specialized persona that guides developers through Domain-Driven Design, sync vs async communication patterns, and data decoupling.

• Extra Asset: Included a standard Architecture Decision Record (ADR) template to document decisions made with the Agent.

Video Demonstration

https://www.loom.com/share/b33d3af2780a4078be0965224c6e3919

Checklist

• Targeted the main branch.
• Passed npm run skill:validate.
• Includes extra assets for both the Skill and Agent.
• Written in English.
• Includes a short video demonstration.

[Copilot]

Pull request overview

This PR adds a new technical-debt-identifier Skill and a new microservices-architect Agent to extend the repository’s reusable Copilot assets, and also normalizes the aws-cloudwatch-investigation skill name to match the repository’s lowercase-hyphenated naming.

Changes:

• Added technical-debt-identifier skill with an example legacy auth snippet under examples/.
• Added microservices-architect agent plus an ADR template asset intended to support architecture decision capture.
• Updated generated docs to include the new skill/agent and adjusted the CloudWatch skill display name.

Reviewed changes

Copilot reviewed 7 out of 7 changed files in this pull request and generated 6 comments.

Show a summary per file

File	Description
skills/technical-debt-identifier/SKILL.md	New skill definition/instructions for generating a structured technical-debt report.
skills/technical-debt-identifier/examples/legacy-auth.js	Example “legacy” code sample intended for demonstrating technical-debt detection.
skills/aws-cloudwatch-investigation/SKILL.md	Renames the skill name to the lowercase-hyphenated form to match conventions.
docs/README.skills.md	Adds the new skill entry and updates the CloudWatch skill entry label.
docs/README.agents.md	Adds the new agent entry.
agents/microservices-architect.agent.md	New agent definition for microservices architecture guidance.
agents/microservices-architect-adr-template.md	ADR template asset intended to accompany the new agent.

[github-actions]

🔒 PR Risk Scan Results

Scanned 6 changed file(s).

Severity	Count
🔴 High	0
🟠 Medium	1
ℹ️ Info	1

Severity	Rule	File	Line	Match
🟠	package-exec-command	docs/README.skills.md	31	| [acreadiness-assess](../skills/acreadiness-assess/SKILL.md)<br />`gh skills install github/awesome-copilot acreadiness-assess` | Run the AgentRC readiness assessment on the curre
ℹ️	skill-script-touched	skills/technical-debt-identifier/examples/legacy-auth.js	1	skills/technical-debt-identifier/examples/legacy-auth.js

This is an automated soft-gate report. Findings indicate review targets and do not block merge by themselves.

[github-actions]

🔍 Vally Lint Results

⚠️ Warnings or advisories found

Scope	Checked
Skills	1
Agents	1
Total	2

Severity	Count
❌ Errors	0
⚠️ Warnings	0
ℹ️ Advisories	1

Summary

Level	Finding
ℹ️	Vally currently lints SKILL.md content. Agent files were detected but skipped:

Full linter output

### Linting skills/technical-debt-identifier
npm warn EBADENGINE Unsupported engine {
npm warn EBADENGINE   package: 'commander@15.0.0',
npm warn EBADENGINE   required: { node: '>=22.12.0' },
npm warn EBADENGINE   current: { node: 'v20.20.2', npm: '10.8.2' }
npm warn EBADENGINE }
npm warn deprecated prebuild-install@7.1.3: No longer maintained. Please contact the author of the relevant native addon; alternatives are available.
✅ technical-debt-identifier (2/2 checks passed)
    ✓ [spec-compliance] All 1 skill(s) are spec-compliant.
        ✓ spec-compliance: All spec checks passed.
    ✓ [valid-refs] All file references across 1 skill(s) are valid.
        ✓ valid-refs: All file references resolve to existing files within the skill directory.

1 skill(s) linted, 1 passed

### Agent files detected (not linted by vally)
ℹ️ Vally currently lints SKILL.md content. Agent files were detected but skipped:
agents/microservices-architect.agent.md