Skip to content

Conversation

@lbussell
Copy link
Member

@lbussell lbussell commented Jan 14, 2026

This PR is targeting #1893, because I wanted to propose changes without completely taking over the PR...

I ran the copilot-instructions file through my create-skill skill, which is based on a bunch of official recommendations from Anthropic. I think what came out the other end is much more clear and concise. I also made a few manual tweaks to it afterwards.

@lbussell lbussell requested a review from a team as a code owner January 14, 2026 18:20
@lbussell lbussell requested a review from mthalman January 14, 2026 18:20
Comment on lines +7 to +10
- **[README.md](../README.md)** - Building the repo locally
- **[src/README.md](../src/README.md)** - Building ImageBuilder container image, available commands
- **[eng/docker-tools/DEV-GUIDE.md](../eng/docker-tools/DEV-GUIDE.md)** - Local development workflows, pipeline architecture, CI/CD patterns
- **[documentation/manifest-file.md](../documentation/manifest-file.md)** - Manifest schema documentation
Copy link
Member

Choose a reason for hiding this comment

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

I'm curious how an agent handles relative paths. Does it keep the context of knowing which directory it got the prompt from so that it can compute these paths relative to that directory? Or is it better to make these paths relative from the root?

1. **ImageBuilder source code** (`src/ImageBuilder/`)
2. **Pipeline templates** (`eng/docker-tools/templates/`)
3. **PowerShell scripts** (`eng/docker-tools/*.ps1`)
4. **DEV-GUIDE.md** - Document changes here!
Copy link
Member

Choose a reason for hiding this comment

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

The new changes don't mention anything about keeping this file in sync.

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