Skip to content

ci: pharn-ci v0.2.0 with test-integrity block - #106

Merged
PrzemekGalarowicz merged 1 commit into
mainfrom
chore/pharn-ci-v0.2.0-block
Oct 25, 2026
Merged

PrzemekGalarowicz merged 1 commit into
mainfrom
chore/pharn-ci-v0.2.0-block

Conversation

@PrzemekGalarowicz

Copy link
Copy Markdown
Member

Moves this repository's pharn-ci check to v0.2.0 and turns on its test-integrity check in block mode. This is the dogfood step of pharn-ci's Epic 1 delivery plan.

Change

  • Pin. pharn-dev/pharn-ci@0873ce23f3595a3497325635b7bf3ee93ea77687 # v0.2.0, replacing v0.1.0, in .github/workflows/ci.yml and the AC-3 workflow test (src/ci/pharn-ci-workflow.ac3.test.ts), which pins the commit.
  • Mode. test-integrity: ${{ github.event_name == 'pull_request' && 'block' || 'warn' }}. The merge queue (merge_group) never reads the accept label, so block mode there would fail a pull request a reviewer has already signed off. Merge-group runs keep warn.

What it does to pull requests here

A PR that deletes a test file, skips or focuses a test, or loosens a test script, a workflow's test step or trigger, or the pharn-ci config fails the check. A reviewer who is not the PR author can sign it off by applying the pharn/accepted label. The workflow's pull_request trigger does not list labeled, so re-run the check after applying the label.

The PR comment and job summary switch to the v0.2.0 layout, with a Test integrity column and a 🔍 Test integrity section. report.json becomes Report v2 (schemaVersion: 2).

Checks run

npm run check:pins passes, src/ci/pharn-ci-workflow.ac3.test.ts has 4 passing tests, and the workflow parses with the new step inputs.

🤖 Generated with Claude Code

- Pin pharn-dev/pharn-ci to the v0.2.0 release commit
  0873ce23f3595a3497325635b7bf3ee93ea77687 (was v0.1.0).
- Set test-integrity: block. A pull request that deletes, skips or focuses a
  test, or loosens a test script, workflow or the pharn-ci config, fails the
  check until a reviewer who is not the PR author applies pharn/accepted.
- Block mode applies on pull_request only. The merge queue never reads the accept label, so
  block there would fail a PR a reviewer already signed off; merge_group runs keep warn.
- src/ci/pharn-ci-workflow.ac3.test.ts pins the new commit.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@coderabbitai

coderabbitai Bot commented Oct 25, 2026

Copy link
Copy Markdown

Important

  • 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

Run configuration
  • Source mode: Auto
  • Repository: Unavailable
  • Branch: Unavailable
  • File: No configuration file from this repository applied
  • Commit: Unavailable
  • Layers: defaults
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 677494bf-8135-4de4-8c2e-db5b11af6287
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@github-actions

Copy link
Copy Markdown

pharn

🛡️ pharn-ci · ⚠️ Warn

Warning

Warn · This change adds no coverable lines; 31 agent setup files found.

Verdict Diff coverage Test integrity Agent setup
⚠️ Warn no coverable lines none (block) 31 files

🧪 Diff coverage

No coverable lines changed.

🔍 Test integrity

No test-integrity findings (mode: block).

💬 Messages (2)

  • ℹ️ NO_COVERABLE_CHANGES No added lines are tracked by the coverage report; nothing to measure.
    • Hint: Nothing to do if the change has no code; otherwise check that the coverage report includes the changed files.
  • ⚠️ CONFIG_CHANGED_IN_DIFF This change edits files that configure this check: `.github/workflows/ci.yml`. Review them before trusting the verdict.
    • Hint: Review the config and workflow changes before trusting the verdict; protect .pharn/ci.yml and .github/workflows/ with CODEOWNERS.

🤖 Agent setup (31 files)

Agent setup: 31 files · claude-code

Agent setup files (31)
Path Kind
.claude/commands/pharn-build.md claude-command
.claude/commands/pharn-grill.md claude-command
.claude/commands/pharn-loop-close.md claude-command
.claude/commands/pharn-loop-quick.md claude-command
.claude/commands/pharn-loop.md claude-command
.claude/commands/pharn-memory-promote.md claude-command
.claude/commands/pharn-plan.md claude-command
.claude/commands/pharn-regress.md claude-command
.claude/commands/pharn-review.md claude-command
.claude/commands/pharn-ship-close.md claude-command
.claude/commands/pharn-ship-quick.md claude-command
.claude/commands/pharn-ship-stages.md claude-command
.claude/commands/pharn-ship.md claude-command
.claude/commands/pharn-spec.md claude-command
.claude/commands/pharn-test.md claude-command
.claude/commands/pharn-verify.md claude-command
.claude/hooks/enforce-writes-scope.cjs claude-hook
.claude/hooks/protect-trusted-paths.cjs claude-hook
.claude/hooks/require-loop-record.cjs claude-hook
.claude/hooks/set-writes-scope.cjs claude-hook
.claude/settings.json claude-settings
.claude/skills/phargus-diagnose claude-skill
.claude/skills/phargus-memory claude-skill
.claude/skills/phargus-rules claude-skill
.claude/skills/phargus-setup claude-skill
.claude/skills/phargus-verify claude-skill
.claude/skills/pharn-ci-cover claude-skill
.claude/skills/pharn-ci-explain claude-skill
.claude/skills/pharn-ci-install claude-skill
.claude/skills/pharn-cli claude-skill
CLAUDE.md claude-md

pharn-ci 0.2.0 · report schema v2 · coverage format lcov

@PrzemekGalarowicz
PrzemekGalarowicz merged commit 66ab740 into main Oct 25, 2026
15 checks passed
@PrzemekGalarowicz
PrzemekGalarowicz deleted the chore/pharn-ci-v0.2.0-block branch October 25, 2026 10:53
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