Skip to content

ci: retire Sentinel Gate in favor of Epic Werkflow, re-enable test CI#159

Merged
jordanpartridge merged 1 commit into
masterfrom
chore/migrate-ci-to-werkflow
Jul 3, 2026
Merged

ci: retire Sentinel Gate in favor of Epic Werkflow, re-enable test CI#159
jordanpartridge merged 1 commit into
masterfrom
chore/migrate-ci-to-werkflow

Conversation

@jordanpartridge

Copy link
Copy Markdown
Contributor

Migrate CI from Sentinel Gate to Epic Werkflow

This repo is now tracked by Epic Werkflow (GitHub App ID 4204113, installed on the conduit-ui org as installation 144093074). This PR moves CI off the standalone "Sentinel Gate" Action and onto the Werkflow pipeline.

What Sentinel Gate did (.github/workflows/gate.yml — removed)

  • Ran synapse-sentinel/gate@v1 (check: certify) on every push/PR to master.
  • Gated merges on a 95% coverage threshold (PCOV).
  • ⚠️ Auto-merged on pass (auto-merge: true, merge-method: squash) — a passing gate would squash-merge the PR automatically, with no human in the loop.

Why it's being retired

Epic Werkflow replaces it. The Werkflow GitHub App posts its own checks (checks:write / statuses:write) once repo runs go through the pipeline, so the standalone gate Action is redundant.

🔴 Notable behavior change: no more auto-merge

Sentinel Gate auto-merged PRs the moment coverage passed. Werkflow's model is the opposite: Jordan reviews every PR himself and nothing merges without him. After this PR, PRs will no longer self-merge on green — that is intentional and the main behavioral change to be aware of.

test.yml.disabled → re-enabled as plain CI (test.yml)

Fate: resurrected as tests-only CI.

Investigation (git history): test.yml was renamed to test.yml.disabled in PR #91 (commit d1aee3a, Jan 19 2026) — the large "AI-first semantic context engine" pivot that rewrote the app from SQLite+Eloquent to a pure Qdrant vector store and deleted 26+ obsolete test files. The workflow was sidelined amid that churn, not because the suite is broken.

The suite is legit and actively maintained today — 85 test files covering the current Qdrant/hybrid-search architecture. Verified it runs green and hermetically (no live Qdrant/Ollama needed; phpunit.xml sets EMBEDDING_PROVIDER=none):

Tests: 5 skipped, 1132 passed (3282 assertions)

So it's brought back as-is: a plain tests-only workflow (composer install + vendor/bin/pest --coverage) on push/PR. It has no gate and no auto-merge — it simply preserves basic CI coverage during the transition. Coverage-based gating is now Werkflow's responsibility, not this workflow's.

Summary

File Change
.github/workflows/gate.yml ❌ Removed (Sentinel Gate + auto-merge → replaced by Werkflow)
.github/workflows/test.yml.disabled ✅ Re-enabled as test.yml (plain tests-only CI)

Do not auto-merge — Jordan reviews.

Remove the synapse-sentinel/gate@v1 "Sentinel Gate" workflow. Coverage
gating and check reporting are now handled by the Epic Werkflow GitHub
App, which posts checks once repo runs go through the pipeline.

Re-enable the previously disabled tests workflow (test.yml.disabled ->
test.yml) as plain CI so basic test coverage is not lost during the
transition. The suite was only sidelined during the Jan 2026 SQLite ->
Qdrant pivot (PR #91); it runs green and hermetically today (1132
passed, 5 skipped) with no live services required.
@jordanpartridge jordanpartridge merged commit f032d48 into master Jul 3, 2026
1 check passed
@jordanpartridge jordanpartridge deleted the chore/migrate-ci-to-werkflow branch July 3, 2026 04:48
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