Skip to content

Fix CI/CD warning noise and Pages deploy false positive - #56

Merged
konard merged 4 commits into
mainfrom
issue-55-7cb0893ca811
Jun 28, 2026
Merged

Fix CI/CD warning noise and Pages deploy false positive#56
konard merged 4 commits into
mainfrom
issue-55-7cb0893ca811

Conversation

@konard

@konard konard commented Jun 28, 2026

Copy link
Copy Markdown
Member

Summary

  • Gates GitHub Pages deployment behind DEPLOY_GITHUB_PAGES=true while still building docs artifacts.
  • Updates workflow actions and Node runtime to remove Node 20 deprecation warnings, with a new CI workflow policy check to prevent regressions.
  • Removes the JS ESLint warning baseline, quiets expected npm release checks, and strips deprecated npm always-auth config before publish commands.
  • Makes JS test targets explicit for Node 24 so CI no longer passes tests/unit/ as a directory to node --test.
  • Adds a patch changeset plus an issue-55 case study with downloaded CI logs, template snapshots, local verification logs, and upstream template reports.

Reproduction Evidence

Template Reports Filed

Verification

  • node scripts/check-ci-workflows.mjs
  • ruby -e 'require "yaml"; Dir[".github/workflows/*.{yml,yaml}"].each { |f| YAML.load_file(f); puts "parsed #{f}" }'
  • npm run lint
  • npm run format:check
  • npm run check:duplication
  • npm run check
  • GITHUB_BASE_REF=main node scripts/validate-changeset.mjs
  • npm test (458 passing)
  • npm run test:coverage (458 passing)
  • npx -y node@24 scripts/run-tests.mjs --reporter spec tests/unit (458 passing under Node v24.18.0)
  • node scripts/run-tests.mjs --env RUN_E2E=true tests/e2e/playwright.e2e.test.js (27 passing)
  • npm run docs:api
  • cargo doc --no-deps --all-features

No UI changes; screenshots are not applicable.

Fixes #55

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

Issue: #55
@konard konard changed the title [WIP] Check for all false positives, warnings and errors in CI/CD and fix them all Fix CI/CD warning noise and Pages deploy false positive Jun 28, 2026
@konard
konard marked this pull request as ready for review June 28, 2026 21:14
@konard

konard commented Jun 28, 2026

Copy link
Copy Markdown
Member Author

Working session summary

Done. PR 56 is updated and ready for review:

#56

Implemented fixes for the CI/CD noise and false positives: guarded Pages deploy, updated action/runtime versions, added workflow policy checks, removed JS lint warning baseline, quieted npm release false-error output, and fixed Node 24 node --test directory handling with an explicit test-file runner.

Fresh CI on HEAD 2d8dbc4 is all passing: CI Workflow Policy, JS, Documentation, Rust, and Python. PR is not draft, branch is 4 commits ahead of main and 0 behind, and the working tree is clean.

Local verification included npm run check, npm test 458/458, npm run test:coverage, Node 24 unit test run 458/458, Playwright e2e smoke 27/27, docs generation, Rust docs, changeset validation, and workflow YAML/policy checks. The case study and preserved logs are in docs/case-studies/issue-55/README.md.


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

@konard

konard commented Jun 28, 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.5
  • Provider: OpenAI
  • Public pricing estimate: $24.517516

📊 Context and tokens usage:

  • 587.3K / 200K (294%) input tokens, 77.1K / 128K (60%) output tokens

Total: (587.3K + 15.2M cached) input tokens, 77.1K output tokens, $24.517516 cost

🤖 Models used:

  • Tool: OpenAI Codex
  • Requested: gpt-5.5
  • Thinking level: max (~31999 tokens)
  • Model: GPT-5.5 (gpt-5.5)

📎 Log file uploaded as Gist (6937KB)


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

@konard
konard merged commit 098c72e into main Jun 28, 2026
30 checks passed
@konard

konard commented Jun 28, 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.

Check for all false positives, warnings and errors in CI/CD and fix them all

1 participant