Skip to content

Conversation

@yoelcommit
Copy link

@yoelcommit yoelcommit commented Jan 29, 2026

Add Demoflow3 with backup prompt for the plan, and security-review skill. Ignore local mcp servers. Update setup instructions and ensure all dependencies are correctly marked. Exclude unnecessary files from version control to streamline the project.
Remove prompts for refactoring that confuse the models when planning the feature.

Copilot AI review requested due to automatic review settings January 29, 2026 13:15
@yoelcommit yoelcommit changed the title Refactor superhero comparison to use backend API Add Demoflow3 Jan 29, 2026
Copy link

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 PR adds documentation and demo workflow files for implementing a superhero comparison API feature. Despite the title suggesting a refactoring implementation, this PR primarily provides planning and documentation materials rather than the actual code implementation.

Changes:

  • Added comprehensive backend API documentation (backend/README.md) describing the planned /api/superheroes/compare endpoint
  • Added DEMOFLOW3.md with step-by-step instructions for an agentic coding workflow demo
  • Added api-security-review skill for automated security reviews of REST APIs
  • Added backup plan document (15-refactor-be-fe-feature-plan.md) and removed two older prompt files (10-FE and 10-BE)
  • Updated .gitignore to exclude .vscode/ directory and mcp.json

Reviewed changes

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

Show a summary per file
File Description
backend/README.md New comprehensive API documentation describing endpoints including a comparison endpoint (not yet implemented in code)
.gitignore Added .vscode/ directory exclusions for local IDE configuration
.github/skills/api-security-review/SKILL.md New security review skill defining checklist and process for API security reviews
.github/prompts/15-refactor-be-fe-feature-plan.md New detailed implementation plan for refactoring comparison feature
.github/prompts/10-FE-refactor-compare-via-api.md Deleted old frontend refactor prompt file
.github/prompts/10-BE-refactor-add-compare-api.md Deleted old backend refactor prompt file
.github/DEMOFLOW3.md New demo workflow document with 11-step agentic coding workflow and best practices

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

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.

1 participant