Repository navigation
Conversation
…ntainer Mounts ~/.eng-central read-only and runs its setup script on container start, so skills, agents and instructions from NHSDigital/eng-central are available inside the container. If eng-central is not set up, an empty folder is mounted and nothing runs.
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.
Description
Makes the shared Copilot skills, agents and instructions from NHSDigital/eng-central available inside this repository's dev container. Part of ENG-1101.
A dev container has its own home folder, so it cannot see the links that eng-central's
scripts/setup.shcreates in~/.copiloton the host. This change:mkdir -p ~/.eng-centralon the host before the container starts (initializeCommand)~/.eng-centralread-only at/opt/eng-central/opt/eng-central/scripts/setup.shon container start (postStartCommand), which links the shared items into the container's~/.copiloteng-central's
setup.shlinks~/.eng-centralto the clone (NHSDigital/eng-central#3), so every repository can mount the same path wherever developers cloned eng-central.If you have not set up eng-central, nothing changes:
~/.eng-centralis an empty folder, so the mount still works, the start command finds no script and does nothing, and the container behaves as before.Testing carried out
Tested in the code-ark dev container:
/opt/eng-central, resolved, and was loaded by Copilot Chat in VS Code inside the containerThe edited
devcontainer.jsonwas validated withdevcontainer read-configuration.QA testing instructions
~/.eng-centraldoes not exist, then run Dev Containers: Rebuild and Reopen in Container. The container starts as before andls ~/.copilotinside it shows nothing from eng-central.mainonce merged) and run./scripts/setup.sh. Check~/.eng-centrallinks to the clone.ls -la ~/.copilot/skillsshows links into/opt/eng-central/skills/./and check the eng-central skills are listed.