Repository navigation
fix(inventory-agentic): working Codespaces/Dev Container setup + GitHub onboarding in Challenge 1 - #608
Merged
Marco Weber (qxsch) merged 3 commits intoNov 9, 2026
Conversation
…ub onboarding in Challenge 1
- Root Codespaces config now uses the Python 3.12 image with Azure CLI, azd and gh,
runs post-create.sh (uv sync + src/.env), forwards port 8000, and resolves paths
via ${localWorkspaceFolderBasename} so uniquely named forks work.
- post-create.sh: runs from any cwd, checks Python 3.11+, installs uv with a
fallback, persists uv on PATH, uses a volume-backed src/.venv (Docker Desktop
bind mounts fail on venv installs).
- .gitattributes: force LF for this hack's and the shared /.devcontainer shell
scripts so local Windows clones do not break post-create.sh / welcome.sh.
- lab-defaults.json: groups ["GHCPUsers"] so lab users get a GitHub account.
- Challenge 1: Azure portal sign-in, GitHub sign-in, fork, Codespace "New with
options" + local Dev Containers path, verification, troubleshooting; with
screenshots reused from the Fraud Intelligence hack.
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
…no GitHub credentials are emitted Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
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.
Fixes the Agentic Inventory Planning hack setup reported by the first delivery team (Fatih Durgut): the Codespace had no Python,
post-create.shwas never called, and Challenge 1 had no GitHub account / fork / Codespaces steps.Root causes
.devcontainer/03-azure-01-04-ai-03-inventory-planning-agentic/devcontainer.json, which was a barebase:ubuntuimage with no Python, no tooling and nopostCreateCommand. The hack-local.devcontainer/(which had the hook) is ignored by Codespaces./workspaces/microhackpaths break for forks with unique names (microhack-labuser-0001).lab-defaults.jsonhadgroups: [], so lab users got no GitHub account.Changes
python:3.12-bookworm+ Azure CLI + azd + gh, runspost-create.sh, forwards port 8000, paths via${localWorkspaceFolderBasename}. Same pattern as Fraud Intelligence.uv(pip, falling back to the astral installer), persists PATH, and uses a volume-backedsrc/.venvbecause Docker Desktop bind mounts fail with I/O errors on venv installs.*.shand the shared/.devcontainer/**/*.sh.welcome.shcurrently fails on every local Windows clone withcore.autocrlf=true."groups": ["GHCPUsers"].images/setup-*.png).Verification
Built the root config with
@devcontainers/cli upon a simulated fork namedmicrohack-labuser-0001(Docker Desktop on Windows):src/.envseeded; postAttachwelcome.shOKaz,azd,ghpresent;PYTHONPATHset; app imports OKuv run uvicorn ui.app:app --port 8000→GET /200 (/api/statereturns 503 untilaz login+ real.envvalues, as documented)Not yet tested in an actual GitHub Codespace. Please have Fatih re-run Challenge 1 there before merging.