Skip to content

Add .github/copilot-instructions.md for cloud agent onboarding#9

Merged
hansec merged 4 commits intocopilot_instructionsfrom
copilot/add-copilot-instructions
Apr 22, 2026
Merged

Add .github/copilot-instructions.md for cloud agent onboarding#9
hansec merged 4 commits intocopilot_instructionsfrom
copilot/add-copilot-instructions

Conversation

Copy link
Copy Markdown

Copilot AI commented Apr 17, 2026

Adds .github/copilot-instructions.md to guide the Copilot cloud agent during onboarding, and updates copilot-setup-steps.yml to align the build configuration.

Primary changes

  • Added .github/copilot-instructions.md with documentation covering the project overview, repository layout, build system, testing, linting, CI workflows, coding conventions, and common pitfalls for the Copilot cloud agent.
  • Updated copilot-setup-steps.yml "Build external libraries" step (renamed from "Build external libraries (OpenMP)"):
    • Removed --oblas_dynamic_arch flag from build_libs.py invocation
    • Added --build_mpich=1 flag to build_libs.py invocation
  • Updated copilot-instructions.md build command to match the action file (removed --oblas_dynamic_arch, added --build_mpich=1)
  • Added a --collect-only example to the testing section of copilot-instructions.md to show how to list available tests in a specific file

Secondary changes

None.

API changes

No API or input file changes.

Copy link
Copy Markdown
Owner

@hansec hansec left a comment

Choose a reason for hiding this comment

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

A couple minor changes to align with actions workflow.

```bash
mkdir builds && cd builds
source ../setup_env.sh # activates Python venv
python ../src/utilities/build_libs.py \
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Update the command line arguments on this to match the updated github action file.

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Updated in f75d3ba — removed --oblas_dynamic_arch and added --build_mpich=1 to match the action file.


```bash
cd builds/build_release/tests
../../run_test.sh physics/test_TokaMaker.py -k "test_name"
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Add an example to collect tests in a specific file as well using the --collect-only flag.

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Added a --collect-only example line in f75d3ba.

Copilot AI requested a review from hansec April 21, 2026 20:06
@hansec hansec changed the base branch from main to copilot_instructions April 22, 2026 04:26
@hansec hansec marked this pull request as ready for review April 22, 2026 04:26
@hansec hansec merged commit 69a568e into copilot_instructions Apr 22, 2026
2 checks passed
@hansec hansec deleted the copilot/add-copilot-instructions branch April 22, 2026 04:27
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