Add Demoflow3#30
Open
yoelcommit wants to merge 19 commits into
Open
Conversation
…feature to use backend API
yoelcommit
changed the title
There was a problem hiding this comment.
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.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
…view and documentation steps
correct typos and improve prompt structure in various markdown files
…l-tweaks Revert "correct typos and improve prompt structure in various markdown files"
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
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.