Skip to content

fix(P0): bind local AMD/Intel and future internal agents symmetrically - #147

Merged
Rafa-Innerchispa merged 1 commit into
mainfrom
chatgpt/p0-local-provider-registry-symmetry-20261012
Oct 12, 2026
Merged

Rafa-Innerchispa merged 1 commit into
mainfrom
chatgpt/p0-local-provider-registry-symmetry-20261012

Conversation

@Rafa-Innerchispa

Copy link
Copy Markdown
Owner

P0: reparar la admisión de proveedores internos AMD / Intel, sin tocar producción

Se verificó en la definición de agent_provider_registry.py que la lista de proveedores internos contenía local-amd-5 (forma con guiones) pero no local-intel-4; sin embargo agent_identity.canonical_mailbox normaliza guiones a guiones bajos, de modo que la admisión por registro no coincidía plenamente con execution_binding.normalize_provider. El binding también utilizaba una lista fija que omitía Intel, impidiendo que futuros proveedores registrados compartieran una sola política.

Corrección aislada

  • Registrar las formas canónicas de ambos proveedores AMD/Intel sin romper las etiquetas existentes.
  • register_internal_provider usa la identidad canónica al registrar proveedores.
  • apply_create_ops_defaults fija local_dev_swarm y permite auto-dispatch local por defecto, respetando una prohibición explícita.
  • execution_binding obtiene los proveedores internos desde el mismo registro, sin lista paralela hardcodeada.
  • Tests reales AMD, Intel y proveedor interno nuevo, incluyendo bloqueo explícito do_not_auto_dispatch=true.

Justificación y aceptación

El canario local ops_db977ed275b8 se ejecutó a las 05:41Z, pero falló correctamente: Qwen describió cambios sin crear archivos objetivo. Temporal marcó failed con Completion prohibited: required evidence missing y objective_paths=[]. Esto confirma que existe un bloqueo en la materialización de cambios del ejecutor local: el presente PR corrige solo la admisión/binding, no afirma corregir esa materialización.

Este PR no modifica vLLM, red, secretos, servicios o main directamente. Exigir CI PASS, inspección del diff y revisión del alcance antes de merge. No reutilizar ni reintentar ops_db977ed275b8 sin reconciliar; conservar su evidencia de fallo y no enviar dos ejecutores al mismo worktree.

La implementación corresponde a la línea P0 preexistente ops_f46a62594c73; no abre otro workflow de coordinación.

@Rafa-Innerchispa
Rafa-Innerchispa marked this pull request as ready for review October 12, 2026 05:44
@Rafa-Innerchispa
Rafa-Innerchispa merged commit 9286519 into main Oct 12, 2026
2 checks passed
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