Skip to content

✨ Set up Copilot instructions#4

Draft
Copilot wants to merge 4 commits into
mainfrom
copilot/setup-copilot-instructions
Draft

✨ Set up Copilot instructions#4
Copilot wants to merge 4 commits into
mainfrom
copilot/setup-copilot-instructions

Conversation

Copy link
Copy Markdown

Copilot AI commented Oct 17, 2025

  • Understand the repository structure and technology stack
  • Identify key project details (Python, FastAPI, Frida, GStreamer)
  • Create .github directory
  • Create copilot-instructions.md file with comprehensive guidelines
  • Review and validate the instructions
  • Address code review feedback (fix placeholder format and grammar)
  • Update instructions to reflect current code organization
  • Add references to main README and other documentation

Summary

Created and updated .github/copilot-instructions.md with comprehensive guidelines for GitHub Copilot.

Latest Updates (addressing feedback)

Updated the Copilot instructions to:

  1. Reference the main README: Added links to README.md in the overview, important notes, and references sections
  2. Reflect current code organization:
    • Documented that the repo has two main components: the main gstaudit app and the girest subproject
    • Clarified that girest is a standalone GIRepository to REST API generator with its own dependencies
    • Noted the different frameworks used (FastAPI for main app, Connexion for girest)
  3. Enhanced dependency documentation: Split dependencies section to show both pyproject.toml files
  4. Improved architectural notes: Better explained why PyGObject isn't used and FastAPI's limitations with references to the README
  5. Added more references: Included OpenAPI to TypeScript and Connexion documentation links

The instructions now accurately represent the dual nature of the project and provide clear references to other documentation.

Original prompt

This section details on the original issue you should resolve

<issue_title>✨ Set up Copilot instructions</issue_title>
<issue_description>Configure instructions for this repository as documented in Best practices for Copilot coding agent in your repository.

</issue_description>

Comments on the Issue (you are @copilot in this section)

Fixes #3


💬 Share your feedback on Copilot coding agent for the chance to win a $200 gift card! Click here to start the survey.

Copilot AI self-assigned this Oct 17, 2025
Copilot AI and others added 2 commits October 17, 2025 16:45
Co-authored-by: turran <24329+turran@users.noreply.github.com>
Co-authored-by: turran <24329+turran@users.noreply.github.com>
Copilot AI changed the title [WIP] Set up instructions for Copilot in repository ✨ Set up Copilot instructions Oct 17, 2025
Copilot AI requested a review from turran October 17, 2025 16:50
@turran
Copy link
Copy Markdown
Contributor

turran commented Oct 20, 2025

@copilot with the current organization of the code, can you update the instructions? Also, can the instructions reference other READMEs?

… references

Co-authored-by: turran <24329+turran@users.noreply.github.com>
Copy link
Copy Markdown
Author

Copilot AI commented Oct 20, 2025

@copilot with the current organization of the code, can you update the instructions? Also, can the instructions reference other READMEs?

Updated the instructions in commit 57219d to reflect the current code organization with:

  • References to the main README throughout (overview, important notes, and references sections)
  • Clear documentation of the two-component structure (main gstaudit app and girest subproject)
  • Separate dependency sections for both pyproject.toml files
  • Notes on the different frameworks used (FastAPI vs Connexion)
  • Links to README sections explaining architectural decisions

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.

✨ Set up Copilot instructions

2 participants