Skip to content

feat(tracing): integrate shared error ownership classifier - #497

Draft
jshaikScale wants to merge 2 commits into
mainfrom
javed/shared-error-ownership-adapter
Draft

feat(tracing): integrate shared error ownership classifier#497
jshaikScale wants to merge 2 commits into
mainfrom
javed/shared-error-ownership-adapter

Conversation

@jshaikScale

@jshaikScale jshaikScale commented Aug 20, 2026

Copy link
Copy Markdown
Contributor

Summary

  • consume the neutral classifier from scale-gp-beta PR #183 through a thin Agentex adapter
  • ignore generic Agentex wrappers/orchestration and own only narrow managed subsystems
  • classify sync, async, and ADK exceptions from their original traceback without changing propagation
  • forward safe category source, classifier version, and deterministic rule ID to SGP

Conservative Agentex policy

Platform-owned prefixes:

  • agentex.lib.core.tracing.processors — Agentex-owned trace export delivery
  • agentex.lib.core.tracing.span_queue — Agentex-owned trace queue/export transport
  • agentex.lib.core.adapters.streams.adapter_redis — managed Redis stream transport and persistence

Ignored prefixes:

  • agentex — generic tracing, orchestration, callback, workflow, and wrapper frames do not establish responsibility
  • sqlalchemy, pyodbc, psycopg, psycopg2, asyncpg, redis — database/client frames defer to the nearest meaningful owner

Specific platform prefixes beat the broad ignored agentex prefix in the shared engine. No package root is globally platform-owned.

Database behavior

  • application/business caller → driver failure = application
  • managed Redis stream persistence → driver failure = platform
  • driver-only or ambiguous traceback = unknown
  • application validation/query construction invoked through a managed wrapper remains application when the application frame or an authoritative typed/explicit signal proves ownership

Dependency status

This PR does not vendor the engine or invent a package version. It remains draft and blocked on merge/publication of the shared SDK API. Standalone CI is expectedly red until that release can be consumed normally.

Validation against shared source

  • 185 passed, 2 skipped — full relevant Agentex tracing, ADK, and processor suite
  • Ruff lint and format checks passed
  • Pyright: 0 errors, 0 warnings, 0 informations
  • focused mypy: Success: no issues found in 2 source files
  • git diff --check passed
Configure Agentex-owned frames in the shared tracing classifier and forward its privacy-safe provenance without duplicating inference logic.

Co-authored-by: Cursor <cursoragent@cursor.com>
@github-actions

Copy link
Copy Markdown

This PR is targeting main, but PRs should target the next branch by default.

The main branch is reserved for release-please and Stainless automation. To resolve, pick one of:

  • Re-target the PR to next (recommended). On the PR page, click Edit next to the title and change the base branch to next.
  • Add the target-main label if this is an intentional exception (e.g. an urgent hotfix). The check will re-run and pass.

See CONTRIBUTING.md for the full branch model.

Ignore generic wrappers and database drivers while reserving platform attribution for managed tracing export and Redis stream persistence paths.

Co-authored-by: Cursor <cursoragent@cursor.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

1 participant