Repository navigation
Rename 96bulls to sessions-pipeline; move setup skill to .agents/skills - #36
Conversation
Rename charts/96bulls -> charts/sessions-pipeline (0.6.0) and charts/96bulls-worker -> charts/sessions-pipeline-worker (0.6.0), the umbrella values key 96bulls: -> sessions-pipeline: (agent-relay 0.3.0), template defines, CI dirs, registry ids, docs and the handoff packager. Move the setup skill from .cursor/skills to .agents/skills. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
|
Important
This repository does not receive automatic reviews because it has fewer than 10 stars.
|
[Medium impact] The reviewed changes appear safe to merge.
|
…he setup skill
Add docs/diagrams/agent-relay-architecture.{mmd,svg,png} (from the prospect
package, telemetry label overlap fixed, single release namespace, preview
components dashed) and agent-relay-request-flow.{mmd,svg,png}, rendered with
mermaid-cli 11.4.2. Link them from the reference architecture and HANDOFF,
require them in the packager, and check them in the handoff smoke.
The setup skill gets license, compatibility and metadata frontmatter, moves
restore and public-verification detail into references/ and the restore
overlay into assets/, and is validated by skills-ref in a new agent-skill
lint job.
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
There was a problem hiding this comment.
All reported issues were addressed
Reply with feedback, questions, or to request a fix.
View guided diff | Turn on auto-fix | Re-trigger cubic
…ilds it charts.sh walks charts alphabetically. charts/96bulls used to run before agent-relay and register the CNPG, MinIO and OTel repositories; renamed to sessions-pipeline it now runs after, so the nested dependency build failed on a clean runner. Register the nested chart's repositories explicitly. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…e from 96bulls - aws-cnpg and gcp-cloudsql CI values mirror the upstream AWS/GCP stacks: keep their Secret (96bulls-runtime, 96bulls-gcp) and bucket (96bulls-ci). - An existing 96bulls umbrella release upgrades in place (names and selectors unchanged); the migration profile uses helm upgrade <existing release>. - README: upgrade section (Secret and runtime role values, standalone worker nameOverride, agent-relay values key). - THIRD_PARTY_NOTICES: current Chart.lock digest; registry comment names the former 96BullSessions repository. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
There was a problem hiding this comment.
All reported issues were addressed
Reply with feedback, questions, or to request a fix.
View guided diff | Turn on auto-fix | Re-trigger cubic
There was a problem hiding this comment.
All reported issues were addressed across 40 files (changes from recent commits).
Requires human review: Auto-approval blocked because this review re-detected 6 unresolved issues already reported by Cubic.
View guided diff | Turn on auto-fix | Re-trigger cubic
- Architecture: no Workload Identity edge to Cloud SQL (Secret-held URLs), Memorystore/Pub/Sub and alleyoop to HoopDb, sandbox namespace <namespace>-<release>, root (dashboard) host, bob-costas stage name, preview hosts only when set, dashed only for preview/optional egress. - Request flow: name every in-cluster call, including relayhistory JWKS. - Skill: full compatibility command list, preview hosts need the overlay, restore overlay also turns off Relayflows (no standalone Job). - Smoke compares the skill against committed blobs and modes. - Reference architecture links the shipped requirements doc; agent-relay 0.2.1 note keeps its historical 96bulls 0.5.5 dependency. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
There was a problem hiding this comment.
All reported issues were addressed across 12 files (changes from recent commits).
Reply with feedback, questions, or to request a fix.
View guided diff | Turn on auto-fix | Re-trigger cubic
Summary
Renames the user-facing "96bulls" name to sessions-pipeline (product name "Sessions Pipeline") across the charts, CI, registry, docs and handoff package. Also:
.agents/skills/and makes it conform to the Agent Skills spec;Charts and versions
charts/96bulls->charts/sessions-pipelinecharts/96bulls-worker->charts/sessions-pipeline-workerpart-oflabel, description and NOTES text changed)96bulls:->sessions-pipeline:(components.sessionsPipelineis unchanged)The renamed charts continue the 0.x line at 0.6.0 instead of restarting at 0.1.0. That keeps them above every published 96bulls version, so nobody mistakes them for older releases.
Upgrade path. An existing
96bullsumbrella release upgrades in place withhelm upgrade <release> agentworkforce/sessions-pipeline. Resource names and selectors are unchanged (the subchart alias is stillworker); only thehelm.sh/chartandpart-oflabels change. The README section "Upgrading a release of the former 96bulls chart" lists the values to keep: Secret names and the runtime role. A standalone96bulls-workerrelease keeps its selectors only withnameOverride: 96bulls-worker.Chart.lock files were regenerated. Template defines are renamed (
96bulls.*->sessions-pipeline.*,96bulls-worker.*->sessions-pipeline-worker.*). Default Secret names, labelapp.kubernetes.io/part-of, the summary-keys mount path, registry ids (opt.sessions-pipeline.*), CI dirs (ci/sessions-pipeline{,-profiles,-search-worker-role,-umbrella}), the workflows,scripts/ci/charts.sh, the packager include list, THIRD_PARTY_NOTICES and the release records are all renamed. The example runtime role is nowsessions_pipeline_runtime, and the docs install the umbrella as releasesessions-pipeline.Default fullnames. No chart in this repo set a
bulls96nameOverride; that came from the upstream Alchemy stack that applied the live proof. Here, names are<release>-<chart>:helm install sessions-pipeline charts/sessions-pipelinegivessessions-pipeline-worker-*andsessions-pipeline-alleyoop-*.helm install sessions-pipeline-worker charts/sessions-pipeline-workergivessessions-pipeline-worker-*.<release>-worker-*and<release>-alleyoop-*. The subchart aliasworkeris unchanged, so umbrella renders are unchanged apart from the chart labels. I checked this by rendering every umbrella, worker and alleyoop CI values file before and after, normalizing the rename, and diffing.The upstream repo
AgentWorkforce/96BullSessionsis nowAgentWorkforce/sessions-pipelineon GitHub (gh repo viewconfirms). Sohome:URLs, upstream citations and the CI checkout now use the new name.Setup skill (Agent Skills standard)
.cursor/skills/setting-up-self-hosted-agent-relay-gcp/to.agents/skills/setting-up-self-hosted-agent-relay-gcp/, with no copy left in.cursor.nameequals the directory, pluslicense: Apache-2.0,compatibility, andmetadata(chart agent-relay, version 0.3.0, standard agentskills.io).references/, and the restore overlay toassets/restore-overlay.yaml(checked to render no Jobs). Content is updated for the current chart layout: Relayflows opt-in, and durable-actors and Relayfile as preview with the actors and files hosts.skills-ref validate(agentskills/agentskills@69ef37e, run on sf-frame):Valid skill. The newagent-skilllint job runs it at that pinned commit, plus test-layout.sh and a no-.cursorcheck.ci/customer-handoff/smoke.shchecks that every tracked skill file ships byte-identical with its exec bit, that no.cursor/ships, and that all six diagram files are present.Diagrams
docs/diagrams/agent-relay-architecture.{mmd,svg,png}builds on the prospect package's diagram with the same structure. Changes:docs/diagrams/agent-relay-request-flow.{mmd,svg,png}is new: Desktop sign-in and device approval, session upload, Dashboard read, and agent-to-agent messaging over Relaycast.-b white -s 2 -w 1800, and the PNGs were inspected.self-hosted-architecture.{svg,png}were re-rendered from the renamed source.docs/self-hosted-reference-architecture.md(new Diagrams section),docs/handoff/HANDOFF.mdanddocs/diagrams/self-hosted-architecture.md.Kept "96bulls" / "bulls96" references (deliberate)
git grep -i -E "96bulls|bulls96"now finds only these:ghcr.io/agentworkforce/96bulls-services, the ECR.../96bulls-services,agentworkforce/96bulls-services(chart defaults, profiles, customer-images.yaml, CI kind tags) and theregistry.example.com/acme/96bulls-servicesfixture. Renaming a published image breaks pulls and digest pins.agentrelay-96bulls-soc2-2610and the bucket, service-account and Artifact Registry names derived from it, plus the packager's split-string denylist for it. These are real cloud resources.charts/sessions-pipeline/ci/aws-cnpg-values.yamlandgcp-cloudsql-values.yamlkeep the Secrets96bulls-runtime/96bulls-gcpand the bucket96bulls-ci, which the upstream AWS/GCP stacks provision. The registry rows match.runtimeRole: bulls96_runtimeinvalues-gcp-proof.yaml/values-gcp-proof-live.yaml(the Cloud SQL role that exists there). The "96bulls namespace" comment invalues-gcp-proof-durable-actors.yaml. Inscripts/selfhost/e2e-gcp-proof.sh: the default--sessions-namespace 96bulls,deploy/bulls96-worker-*, thebulls96-alleyoop-ingestprobe comment, and the deprecated--skip-96bullsalias.sessions.__96bulls_migrations, the role markersessions.__96bulls_role_validation, and the alleyoop issuer defaulthttps://96bulls.local, which matches the services' built-inAUTH_ISSUERdefault. Env var names (HOOPDB_*,LOCKER_*, ...) are unchanged.BULLS96_READ_TOKEN. Renaming it needs a repo-settings change first, so it can follow in a later PR.compatibility.releasesrows96bulls-0.5.4/0.5.2/0.5.1. The registry checks these against the real git tags96bulls-*/96bulls-worker-*. Their localsourcesnow point at the tag commits.AgentWorkforce/96BullSessions").docs/self-hosted-requirements.mdrows rendered from those.Test plan
Run locally on this branch:
scripts/ci/charts.sh staticci/agent-relay/render.shscripts/ci/deps.shci/sessions-pipeline-search-worker-role/render.shci/customer-handoff/smoke.shci/sessions-pipeline-profiles/matrix.sh staticnode ci/self-hosted-requirements/registry.mjs check, andrenderwith no diffci/self-hosted-requirements/test.shpython3 ci/customer-release-gate/test.pyvalidate-customer-release.py --consistency(evidenceimageManifestSha256updated for the customer-images.yaml rename)scripts/ci/version-bump.sh origin/maintest-layout.shskills-ref validatereportsValid skillcharts.sh staticwith an empty Helm repo config (reproduces the clean-runner failure fixed in a61c9a2)charts.sh dryrun, matrix dryrun, search-worker-role kind) run in CI.🤖 Generated with Claude Code