Skip to content

fix(ci): honor excluded paths for all events - #114

Merged
konard merged 4 commits into
mainfrom
issue-113-4d934d95a1c7
Jul 27, 2026
Merged

fix(ci): honor excluded paths for all events#114
konard merged 4 commits into
mainfrom
issue-113-4d934d95a1c7

Conversation

@konard

@konard konard commented Jul 27, 2026

Copy link
Copy Markdown
Member

Fixes #113

Summary

  • filter ignored path prefixes before computing every change-gating output
  • remove unconditional push bypasses from change-gated jobs
  • consolidate JavaScript detection and remove redundant detector outputs
  • cover pull request and push events for experiments/, dev/log/, and docs/case-studies/

Reproduction

Before this change, a direct push containing only an excluded file still ran lint, tests, and other gated jobs because each condition included an unconditional push alternative. On pull requests, excluded .mjs, .js, and .md files could independently set extension outputs before the exclusion policy was applied.

The new regression fixtures create excluded-only commits for both pull request synthetic merges and direct pushes. They verify that every detector output remains false and that all change-gated workflow jobs evaluate to skipped.

Verification

  • node --test --test-timeout=30000 tests/detect-code-changes.test.js tests/workflow-reliability.test.js
  • npm test
  • npm run check
  • bash scripts/check-mjs-syntax.sh
  • bash scripts/check-file-line-limits.sh
  • GITHUB_BASE_REF=main node scripts/validate-changeset.mjs

Adding .gitkeep for PR creation (default mode).
This file will be removed when the task is complete.

Issue: #113
@konard konard self-assigned this Jul 27, 2026
@konard konard changed the title [WIP] CI: excluded paths bypass change detection on direct pushes fix(ci): honor excluded paths for all events Jul 27, 2026
@konard
konard marked this pull request as ready for review July 27, 2026 04:12
@konard

konard commented Jul 27, 2026

Copy link
Copy Markdown
Member Author

Working session summary

Implemented and finalized PR #114.

Key changes:

  • Applied one ignored-path policy before computing all CI outputs.
  • Removed unconditional push bypasses from change-gated jobs.
  • Consolidated JavaScript detection and removed redundant outputs.
  • Added PR/push regression coverage for experiments/, dev/log/, and docs/case-studies/.
  • Added positive coverage ensuring normal source and documentation changes remain detected.
  • Added a patch changeset.

Verification:

  • Full local tests and checks pass.
  • All nine Node/Bun/Deno × Linux/macOS/Windows CI jobs pass.
  • Link checking, linting, changeset validation, syntax, and line-limit checks pass.
  • PR is ready for review, mergeable, and the working tree is clean.

This summary was automatically extracted from the AI working session output.

@konard

konard commented Jul 27, 2026

Copy link
Copy Markdown
Member Author

🤖 Solution Draft Log

This log file contains the complete execution trace of the AI solution draft process.

💰 Cost estimation:

  • Model: GPT-5.6 Sol
  • Provider: OpenAI
  • Public pricing estimate: $4.888720

📊 Context and tokens usage:

  • 164.4K / 200K (82%) input tokens, 12.2K / 128K (10%) output tokens

Total: (164.4K + 2.7M cached) input tokens, 12.2K output tokens, $4.888720 cost

🤖 Models used:

  • Tool: OpenAI Codex
  • Requested: gpt-5.6-sol
  • Thinking level: low (~7999 tokens)
  • Model: GPT 5.6 Sol (gpt-5.6-sol)

📎 Log file uploaded as Gist (1526KB)


Now working session is ended, feel free to review and add any feedback on the solution draft.

@konard
konard merged commit 2bafdd9 into main Jul 27, 2026
22 checks passed
@konard

konard commented Jul 27, 2026

Copy link
Copy Markdown
Member Author

🎉 Auto-merged

This pull request has been automatically merged by hive-mind.

  • All CI checks have passed

Auto-merged by hive-mind with --auto-merge flag

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.

CI: excluded paths bypass change detection on direct pushes

1 participant