Skip to content

Change integrated prompts to -apm suffix#30

Merged
danielmeppiel merged 3 commits intomainfrom
clean-main
Nov 14, 2025
Merged

Change integrated prompts to -apm suffix#30
danielmeppiel merged 3 commits intomainfrom
clean-main

Conversation

@danielmeppiel
Copy link
Copy Markdown
Collaborator

This pull request updates the prompt integration naming convention from an @ prefix to a -apm suffix, improving discoverability and avoiding conflicts in VSCode and Copilot. The change affects prompt file generation, .gitignore patterns, documentation, and all related tests to ensure a consistent and intent-first naming approach.

Copilot AI review requested due to automatic review settings November 14, 2025 13:30
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This pull request migrates the prompt integration naming convention from an @ prefix (e.g., @accessibility-audit.prompt.md) to a -apm suffix (e.g., accessibility-audit-apm.prompt.md). The change improves VSCode autocomplete discoverability and avoids symbol conflicts with Copilot's @ agent syntax.

Key Changes:

  • Updated core filename generation logic in PromptIntegrator.get_target_filename() to use -apm suffix pattern
  • Modified .gitignore pattern from @*.prompt.md to *-apm.prompt.md
  • Added comprehensive test coverage for the new suffix pattern with edge case validation

Reviewed Changes

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

Show a summary per file
File Description
pyproject.toml Bumped version from 0.5.2 to 0.5.3
uv.lock Updated package version (inconsistent with pyproject.toml)
src/apm_cli/integration/prompt_integrator.py Implemented -apm suffix logic in get_target_filename() and updated gitignore pattern
src/apm_cli/cli.py Enhanced user feedback message to show the new pattern
tests/unit/integration/test_prompt_integrator.py Updated all test assertions and added TestPromptSuffixPattern class for edge cases
tests/integration/test_auto_integration.py Updated integration test assertions to use new suffix pattern
docs/integrations.md Partially updated documentation with new pattern and intent-first discovery explanation
docs/cli-reference.md Not updated - still references old @ prefix pattern
CHANGELOG.md Added comprehensive release notes with migration guidance
Comments suppressed due to low confidence (1)

docs/integrations.md:289

  • Comment still references the old @*.prompt.md pattern. Update to *-apm.prompt.md to match the new naming convention.
# .github/prompts/@*.prompt.md (integrated prompts)

@danielmeppiel danielmeppiel merged commit 6f2f363 into main Nov 14, 2025
22 checks passed
@danielmeppiel danielmeppiel deleted the clean-main branch February 27, 2026 09:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants