Skip to content

build: exclude local credentials from SenseVoice image context - #353

Merged
LauraGPT merged 1 commit into
mainfrom
codex/sensevoice-container-context-20260910
Sep 10, 2026
Merged

build: exclude local credentials from SenseVoice image context#353
LauraGPT merged 1 commit into
mainfrom
codex/sensevoice-container-context-20260910

Conversation

@LauraGPT

Copy link
Copy Markdown
Member

Summary

  • Add a Docker build-context boundary for Git metadata and common local credential files before COPY . /app.
  • Disable persisted checkout credentials in both container jobs and trigger the workflow when the ignore rules or context probe change.
  • Exercise Docker/BuildKit's actual context matcher with harmless sentinels and verify that required runtime files remain available before the full image build.

Verification

  • New regression tests first failed on the missing ignore file and workflow protections.
  • 14 container/device contract tests pass locally; shell syntax and git diff checks pass.
  • Read-only review found no P1/P2 issues.
  • Docker is not installed on the authorized development host, so the real BuildKit context probe and full image build must pass on the exact PR head in CI. This is not a local image/inference acceptance claim.

The GHCR package remains private. This change does not make anonymous pulls work, change visibility, invalidate published tags, or claim that an actual secret leak was observed. The ignore rules cover common names, not arbitrary secrets embedded in source files. Existing runtime behavior is unchanged.

Signed-off-by: LauraGPT <18321252+LauraGPT@users.noreply.github.com>
@LauraGPT
LauraGPT merged commit 858d34e into main Sep 10, 2026
2 checks passed
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.

1 participant