Repository navigation
Replies: 1 comment 2 replies
|
2 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Uh oh!
There was an error while loading. Please reload this page.
I want to lock down scope before starting, since it changes what actually gets
built:
Scope: should the first PR land as a general
tools/jev/adapterusable by any skill, or scoped narrowly to
pr-management-triagefirst,with wider adoption gated on eval results (see below)? I'd lean toward
the narrow pilot so we're not maintaining a general-purpose tool before
a single skill has proven it out.
RFC: does this need a new RFC given it touches vendor neutrality,
or is a tool adapter + per-skill PR enough, treated like any other
opt-in
tools/<system>/adapter? My inclination is the latter, providedthe adapter is fully optional and every skill degrades to its current
behavior with it disabled — but I'd like a maintainer/PMC read on this
before investing in code.
Bar for adoption: assuming (1) is the pilot route, what would
count as a clear yes/no signal? I'm planning to run a held-out sample of
already-labeled PRs from
pr-management-triage's history through Jevand report agreement rate, p50/p95 latency, and cost per 100 calls back
here. Flag if there's a different metric or sample size you'd want to
see before considering wider rollout.
In the meantime I'll start on the adapter itself (dependency-free HTTP
client, pinned model version, fail-open on any error/timeout/low
confidence) since that part is needed regardless of how (1)/(2) land — I'll
hold off on wiring it into any skill until there's agreement here.
All reactions