ci: drop the axiom pipeline telemetry - #108
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Close or redirect — Telemetry removal is premature without auto-merge or replacement alertingThis PR should be closed or redirected because its stated replacement is absent:
Claims that did not hold
Blocking
Merge state: The PR is open and mergeable. The live check rollup has 8 completed checks (7 success and 1 neutral) and 1 in progress. The human review summary is empty; unresolved review-thread state is unknown because no thread data was supplied. Evidence · 3 verified · 2 not verifiedVerified
Not verified
Read the full diff and both changed workflows, the fetch/change support scripts, package manifests,
Comment Review by Rupic. |
Deploying with
|
| Status | Name | Latest Commit | Preview URL | Updated (UTC) |
|---|---|---|---|---|
| ✅ Deployment successful! View logs |
modelpedia-api | 8c0647b | Commit Preview URL Branch Preview URL |
Aug 31 2026, 07:23 AM |
problem
the axiom pipeline telemetry (#103, #107) monitored the symptom: it would page when the daily data PR sat unmerged past 48h, but a human still had to go merge it. the direction is now to remove the human step entirely (auto-merge logic, likely via rupic, with pricing sanity moved into
validate.tsas a merge gate), which makes the staleness monitoring layer moot before its dataset was ever provisioned.change
remove both axiom reporting steps (
pipeline.run/pipeline.providerin fetch-models,model.changein changes). the Better Stack heartbeat stays: it answers "is the pipeline alive at all", which auto-merge logic depends on too and cannot report on its own.verification
yaml lints clean; both steps were no-ops in production anyway (the
AXIOM_TOKENsecret was never provisioned, so every run skipped them).