…ignment

- Add Invoke-CopilotAssignment.ps1 script for synthesizing context from trusted sources
- Add copilot-synthesis.yml config with maintainers and AI agents configuration
- Add Pester tests (60 tests) for the synthesis script
- Update SKILL.md with documentation for copilot operations

Implements Issue #92 Phase 1

Co-authored-by: rjmurillo <6811113+rjmurillo@users.noreply.github.com>

Co-authored-by: rjmurillo <6811113+rjmurillo@users.noreply.github.com>

- Move reusable functions to GitHubHelpers.psm1:
  - Get-IssueComments, Update-IssueComment, New-IssueComment
  - Get-TrustedSourceComments (useful for pr-comment-responder)
- Move script from scripts/copilot/ to scripts/issue/ (no nested dir)
- Move config from .github/ to .claude/skills/github/ (app config)
- Use config extraction patterns in Get-CodeRabbitPlan
- Fix timestamp to use Get-Date -AsUTC
- Remove unused $IssueTitle parameter from New-SynthesisComment
- Fix regex efficiency (remove redundant Where-Object)
- Update tests to reflect new structure

Addresses PR review feedback from @rjmurillo

Co-authored-by: rjmurillo <6811113+rjmurillo@users.noreply.github.com>

- Fix YAML parsing bug: synthesis marker regex now specifically targets
  'synthesis:' section to avoid matching 'ai_triage.marker' first
- Add 29 functional tests that execute functions and verify behavior:
  - Get-MaintainerGuidance: empty input, bullet extraction, checkboxes
  - Get-CodeRabbitPlan: implementation extraction, issue references
  - Get-AITriageInfo: priority/category extraction
  - Find-ExistingSynthesis: idempotency marker detection
  - New-SynthesisComment: comment generation with all sections
  - Get-SynthesisConfig: YAML parsing and marker extraction

Total: 90 tests pass (61 pattern-based + 29 functional)

Co-authored-by: rjmurillo-bot <250269933+rjmurillo-bot@users.noreply.github.com>

- Use deep copy for config hashtable to avoid modifying defaults
- Wrap ForEach-Object results in @() to ensure arrays (not null)
- Use JSON payload via --input for large comment bodies in both
  Update-IssueComment and New-IssueComment functions

All 146 tests pass.

Co-authored-by: rjmurillo <6811113+rjmurillo@users.noreply.github.com>

…nment

Addresses cursor[bot] review comments:

- Fix module path: Use "../.." to navigate from scripts/issue/ to modules/
  (was using only ".." which resolved to non-existent scripts/modules/)
- Fix corrupted Unicode: Replace corrupted emoji in related_prs pattern
  with correct link emoji

Comment-ID: 2637061449, 2637061451

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

Add PR reviewers to ai_agents trusted sources:
- Copilot (GitHub Copilot)
- cursor[bot] (Cursor Bugbot)

Updated both config file and script defaults.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

Add comprehensive inline documentation covering:
- Purpose and overview of the synthesis system
- Usage examples and related files
- Detailed explanation of trusted_sources section
- Description of each AI agent and their contributions
- Extraction patterns with examples
- Synthesis marker and idempotency behavior
- Extension guide for adding maintainers, agents, and patterns

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

rjmurillo-bot is a bot account, so it belongs in ai_agents rather than
maintainers. Updated config, script defaults, and tests to match.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

Implements the GitHub Actions workflow for automated Copilot context
synthesis and assignment when the 'copilot-ready' label is added.

Changes:
- Create .github/workflows/copilot-context-synthesis.yml workflow
  - Triggers on issues:labeled event with 'copilot-ready' gate
  - Supports workflow_dispatch for manual testing
  - Generates step summary on success
- Fix empty comments handling in Invoke-CopilotAssignment.ps1
  - Wrap comments in @() to ensure array even if null
- Fix Get-TrustedSourceComments in GitHubHelpers.psm1
  - Add [AllowEmptyCollection()] attribute
  - Return empty array early for empty input

Addresses all acceptance criteria from Issue #92 Phase 4.

Fixes #92

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

- Add step to remove the copilot-ready label after successful processing
- Only removes label for labeled trigger (not workflow_dispatch)
- Update step summary to reflect label removal action

Also created the copilot-ready label:
- Color: #6e40c9 (GitHub Copilot purple)
- Description: Triggers automated context synthesis and Copilot assignment

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

Add a second job that runs hourly to catch any missed issues:

- schedule: Runs every hour via cron "0 * * * *"
- sweep-missed job: Lists all open issues with copilot-ready label
- Processes each issue using the same Invoke-CopilotAssignment.ps1 (DRY)
- Removes label after successful processing
- Continues processing even if individual issues fail

Smart workflow_dispatch modes:
- With issue_number: Process single issue
- Without issue_number: Run sweep mode

Both jobs use the same PowerShell script as the single source of truth,
ensuring consistent behavior across all trigger types.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

Comprehensive analysis of 56 review comments:
- 47 addressed (84%), 9 pending (16%)
- P0 (Blocking): 0 - All resolved
- P1 (High): 2 items (functional tests complete)
- P2 (Medium): 4 items (deferred to follow-up)
- P3 (Low): 3 items (cosmetic)

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

Session 38: Executed action plan for PR #147 review feedback
- Verified 90/90 tests pass (including functional tests)
- Posted summary comment and re-requested review from rjmurillo
- All P1 items complete, P2-P3 deferred to follow-up

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

- Use non-greedy regex quantifier with boundary for maintainers/ai_agents
  extraction to prevent over-matching in YAML parsing (P2-001)
- Add RelatedPRs to hasAIContent check so AI recommendations section
  displays when only related PRs exist (P2-002)

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

Define where test files should be located:
- All Pester tests in /tests/ directory at repo root
- Naming convention: {ScriptName}.Tests.ps1
- Pattern-based vs functional test organization

Addresses P2-003 from PR #147 review feedback.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

Define JSON Schema for copilot-synthesis.yml covering:
- trusted_sources (maintainers, ai_agents)
- extraction_patterns (coderabbit, ai_triage)
- synthesis (marker)

Enables IDE validation and autocompletion for config files.
Addresses P2-004 from PR #147 review feedback.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

Add edge case test coverage for:
- Empty and malformed config file handling
- Multiple maintainer comments order preservation
- Unicode character handling in patterns
- RelatedPRs visibility in AI section

Add test approach documentation explaining:
- Pattern-based tests for structural validation
- Functional tests for behavior verification
- Why both approaches are used together

Addresses P3-001 and P3-003 from PR #147 review feedback.
Total tests: 100

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

Import 12 learnings from CodeRabbit AI export, validating each for
atomicity and checking against existing skills for duplicates.

Summary:
- 8 valid skills created (7 new + 1 supplementary)
- 4 duplicates identified (already in skills-linting)
- Cross-reference added to skills-linting

Skills added:
- Skill-CodeRabbit-001: MCP tool path case sensitivity (95%)
- Skill-CodeRabbit-002: Template bracket notation placeholders (93%)
- Skill-CodeRabbit-003: Infrastructure naming avoids spaces (90%)
- Skill-CodeRabbit-004: Expression injection labeling is intentional (95%)
- Skill-CodeRabbit-005: MCP tool naming with duplicated segments (92%)
- Skill-CodeRabbit-006: Generated files omit edit warnings (90%)
- Skill-CodeRabbit-007: Analyst vs impact analysis architecture (95%)
- Skill-CodeRabbit-008: Nested code fence syntax (88%)

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

- Take main's regex fix for synthesis marker extraction
- Take main's HANDOFF.md and test file (more current)