Skip to content

Remove docker-sbx runtime from Daily Dashboard Language Spec Review workflow - #58

Merged
pelikhan merged 2 commits into
mainfrom
copilot/aw-configure-docker-sbx-secrets
Aug 27, 2026
Merged

Remove docker-sbx runtime from Daily Dashboard Language Spec Review workflow#58
pelikhan merged 2 commits into
mainfrom
copilot/aw-configure-docker-sbx-secrets

Conversation

Copilot AI commented Aug 27, 2026

Copy link
Copy Markdown
Contributor

The Daily Dashboard Language Specification Review workflow was configured for docker-sbx, which hard-required Docker Hub credentials and caused runs to skip when DOCKER_USERNAME/DOCKER_PAT were not present. This PR switches the workflow off docker-sbx so it can run with the repository’s standard agent runtime and no Docker Hub secret dependency.

  • Workflow source update

    • Removed the sandbox.agent.runtime: docker-sbx block from:
      • .github/workflows/daily-dashboard-language-spec-review.md
  • Compiled workflow lock alignment

    • Updated .github/workflows/daily-dashboard-language-spec-review.lock.yml to match source intent:
      • Removed Docker Hub secret declarations (DOCKER_PAT, DOCKER_USERNAME)
      • Removed docker-sbx-specific setup/check/install/auth steps
      • Removed docker-sbx-specific job gating and failure wiring
  • Contract coverage

    • Added focused unit assertions in tests/unit/workflow-contract.test.mjs to enforce:
      • no runtime: docker-sbx in source
      • no docker-sbx / Docker Hub secret references in the lock file
# before
sandbox:
  agent:
    id: awf
    runtime: docker-sbx

# after
# (sandbox runtime block removed)

Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Copilot AI changed the title [WIP] Add Docker Hub secrets for docker-sbx in daily dashboard workflow Remove docker-sbx runtime from Daily Dashboard Language Spec Review workflow Aug 27, 2026
Copilot AI requested a review from pelikhan August 27, 2026 23:19
@pelikhan
pelikhan marked this pull request as ready for review August 27, 2026 23:29
@pelikhan
pelikhan merged commit 17e7449 into main Aug 27, 2026
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.

[aw] Daily Dashboard Language Specification Review is missing docker-sbx Docker Hub secrets

2 participants