diff --git a/.github/workflows/opencode-review.yml b/.github/workflows/opencode-review.yml index 6ae26d68d..f3d93afa8 100644 --- a/.github/workflows/opencode-review.yml +++ b/.github/workflows/opencode-review.yml @@ -653,15 +653,10 @@ jobs: while IFS= read -r dockerfile; do [ -n "$dockerfile" ] || continue context_dir="$(dirname "$dockerfile")" - if [ "$context_dir" = "." ]; then - dockerfile_arg="$dockerfile" - else - dockerfile_arg="$(basename "$dockerfile")" - fi tag_suffix="$(printf '%s' "$dockerfile" | tr '[:upper:]' '[:lower:]' | tr '/.' '--' | tr -cd '[:alnum:]-' | cut -c1-80)" image_tag="opencode-review-${PR_HEAD_SHA:-head}-${tag_suffix}" run_and_capture "Docker build (${dockerfile})" \ - docker build --pull=false -f "$dockerfile_arg" -t "$image_tag" "$context_dir" + docker build --pull=false -f "$dockerfile" -t "$image_tag" "$context_dir" done < <(git ls-files 'Dockerfile' '*/Dockerfile' 'Dockerfile.*' '*/Dockerfile.*') if has_changed_tracked_files 'docker-compose.yml' 'docker-compose.yaml' 'compose.yml' 'compose.yaml'; then for compose_file in docker-compose.yml docker-compose.yaml compose.yml compose.yaml; do diff --git a/scripts/ci/test_strix_quick_gate.sh b/scripts/ci/test_strix_quick_gate.sh index 363431591..d5a6d5f68 100755 --- a/scripts/ci/test_strix_quick_gate.sh +++ b/scripts/ci/test_strix_quick_gate.sh @@ -648,7 +648,7 @@ assert_opencode_review_uses_codegraph_and_gpt5_fallback() { assert_file_contains "$workflow_file" "check:python-docstrings" "opencode coverage evidence can use repository Python docstring gates exposed through package scripts" assert_file_contains "$workflow_file" "Coverage execution evidence" "opencode evidence exposes coverage measurement to the review model" assert_file_contains "$workflow_file" 'context_dir="$(dirname "$dockerfile")"' "opencode Docker evidence builds each Dockerfile from its containing directory" - assert_file_contains "$workflow_file" 'docker build --pull=false -f "$dockerfile_arg" -t "$image_tag" "$context_dir"' "opencode Docker evidence uses a context-relative Dockerfile argument" + assert_file_contains "$workflow_file" 'docker build --pull=false -f "$dockerfile" -t "$image_tag" "$context_dir"' "opencode Docker evidence keeps Dockerfile paths relative to the source checkout" assert_file_contains "$workflow_file" "has_changed_tracked_files 'docker-compose.yml' 'docker-compose.yaml' 'compose.yml' 'compose.yaml'" "opencode Docker evidence runs compose checks only when compose files changed" assert_file_contains "$workflow_file" "Coverage and Docstring coverage labels must cite Coverage execution evidence showing supported repository test suites passed" "opencode approval requires passing test evidence when coverage is applicable" assert_file_contains "$workflow_file" "or explicitly cite Coverage execution evidence as not applicable because no supported source files or package manifests were found" "opencode approval permits only evidence-backed no-source coverage N/A"