Skip to content

fix(demo): copy hatch_build.py into the image build context - #474

Merged
plind-junior merged 1 commit into
mainfrom
fix/demo-hatch-build
Jul 13, 2026
Merged

fix(demo): copy hatch_build.py into the image build context#474
plind-junior merged 1 commit into
mainfrom
fix/demo-hatch-build

Conversation

@plind-junior

@plind-junior plind-junior commented Jul 13, 2026

Copy link
Copy Markdown
Member

the demo image build broke when pyproject.toml gained the hatch_build.py build hook (console bundling): pip install of /src fails before dependency resolution because the hook file was never copied into the build context. one-line copy fixes it; verified by rebuilding the image locally.

Summary by CodeRabbit

  • Bug Fixes
    • Fixed demo container builds by ensuring required packaging configuration is available during dependency installation.

pyproject.toml declares hatch_build.py as a build hook (the conditional
console bundling added for vouch console), so pip install of /src fails
before dependency resolution when the file is absent. the demo image
has been unbuildable since that hook landed.
@coderabbitai

coderabbitai Bot commented Jul 13, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: 3bb2893f-ab56-4fa8-b9ba-b4b1ecbce675

📥 Commits

Reviewing files that changed from the base of the PR and between 5363642 and 15ac67a.

📒 Files selected for processing (1)
  • demo/Dockerfile

📝 Walkthrough

Walkthrough

The runtime Docker stage now copies hatch_build.py into /src/ with pyproject.toml and README.md, allowing the sdist build backend to resolve its build hook.

Changes

Docker build preparation

Layer / File(s) Summary
Include Hatch build hook
demo/Dockerfile
The runtime-stage source copy includes hatch_build.py and explains why it is required before dependency resolution.

Estimated code review effort: 1 (Trivial) | ~2 minutes

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly describes the main fix: adding hatch_build.py to the demo image build context.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/demo-hatch-build

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@github-actions github-actions Bot added the size: XS less than 50 changed non-doc lines label Jul 13, 2026
@plind-junior
plind-junior merged commit f1826ac into main Jul 13, 2026
12 checks passed
jsdevninja pushed a commit to jsdevninja/vouch that referenced this pull request Jul 15, 2026
entries for the audit-ordering + cli-parity fix (vouchdev#475), the unified
search path with recency and degradation honesty (vouchdev#476), and the demo
image build fix (vouchdev#474).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size: XS less than 50 changed non-doc lines

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant