Skip to content

Commit 6298c12

Browse files
committed
docs: restore and update tracking documents for upstream-reset state
https://claude.ai/code/session_01W9xh1kJCqtHqBgBAMQikon
1 parent 81eddf4 commit 6298c12

3 files changed

Lines changed: 81 additions & 0 deletions

File tree

docs/PENDIENTES_DE_EMPEZAR.md

Lines changed: 29 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,29 @@
1+
# Pendientes de empezar (sin comenzar)
2+
3+
> Fecha: junio 2026. Cobertura global actual: **69,3 %** (sobre `main` = upstream).
4+
5+
## Paquetes con menor cobertura (objetivos)
6+
7+
| Paquete | Cobertura | Notas |
8+
|---|---:|---|
9+
| `internal/ghmcp` | **0,0 %** | Arranque del servidor, hosts GHES/GHEC, transporte HTTP. Mayor hueco e impacto. |
10+
| `cmd/mcpcurl` | ~55 % | **En progreso** (este trabajo). Resto sin cubrir: rutas con subproceso (`executeServerCommand`, `Run`). |
11+
| `internal/githubv4mock` | 18,1 % | Utilidad de test; cobertura indirecta. |
12+
| `pkg/http` | 37,4 % | Capa del servidor HTTP remoto. |
13+
| `pkg/http/transport` | 46,7 % | Transporte HTTP. |
14+
| `pkg/log` | 47,4 % | Logging E/S. |
15+
| `pkg/http/middleware` | 50,6 % | Middleware del servidor. |
16+
| `pkg/utils` | 54,0 % | Helpers de resultados. |
17+
18+
## Tareas concretas sin empezar
19+
1. **`internal/ghmcp`**: tests de derivación de hosts GHES/GHEC, wrappers de
20+
transporte/`RoundTrip`, middleware. Mayor impacto en el total.
21+
2. **`pkg/http` y submódulos** (`transport`, `middleware`, `oauth`): rutas del
22+
servidor remoto.
23+
3. **`cmd/mcpcurl`** (rematar): cubrir `executeServerCommand` y el `Run` de los
24+
comandos generados (requiere un binario/servidor de prueba por stdio).
25+
26+
## Recomendación de orden
27+
Confirmar primero la estabilidad vs upstream (ver PENDIENTES_DE_RESOLVER). Si se
28+
mantiene: `internal/ghmcp``pkg/http*`. Re-medir la cobertura del paquete antes
29+
de cada tarea, porque el repo cambia con frecuencia.

docs/PENDIENTES_DE_RESOLVER.md

Lines changed: 18 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,18 @@
1+
# Pendientes de resolver (en curso / decisión pendiente)
2+
3+
> Fecha: junio 2026.
4+
5+
## 1. ¿Abrir PR con los tests de `cmd/mcpcurl`?
6+
- Estado: el fichero `cmd/mcpcurl/coverage_test.go` está hecho y verde en local,
7+
commiteado en la rama. Falta decidir si se abre PR contra `main`.
8+
9+
## 2. Estabilidad del repo frente al upstream — RIESGO
10+
- `main` ya se ha reseteado al upstream una vez, borrando trabajo previo.
11+
- Mientras no se aclare la política del fork (¿se sigue sincronizando con
12+
`github/github-mcp-server`?), **cualquier test nuevo puede volver a perderse**.
13+
- Pendiente: confirmar con el responsable si el fork va a divergir del upstream
14+
o seguir sincronizándose, para decidir cuánto esfuerzo invertir.
15+
16+
## 3. Documentos de seguimiento
17+
- Recuperados en `docs/` y puestos al día (este conjunto). Pendiente: mantenerlos
18+
vivos según avance el trabajo.

docs/TRABAJO_REALIZADO.md

Lines changed: 34 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,34 @@
1+
# Trabajo realizado
2+
3+
> Proyecto: **github-mcp-server** (servidor MCP en Go). Fecha: junio 2026.
4+
> Rama de trabajo: `claude/test-coverage-analysis-s33ri0`.
5+
6+
## Nota de contexto importante
7+
El `main` del fork se **sincronizó/reseteó con el upstream oficial
8+
`github/github-mcp-server`**. Eso **eliminó de `main`** todo el trabajo previo
9+
del fork (tests de `pkg/lockdown` del PR #5, documentos de estado, los arreglos
10+
de workflows, etc.). El historial actual de `main` es el del upstream. Por eso
11+
este documento se ha reescrito para reflejar la realidad actual.
12+
13+
## Histórico (referencia)
14+
- Análisis inicial de cobertura del repo.
15+
- **PR #5** (fusionado en su momento): tests de `pkg/lockdown` → perdidos en el reset.
16+
- **PR #10** (cerrado sin fusionar): arreglo de CI mezclado con docs.
17+
- **PR #12** (cerrado): reintento de arreglo de CI; quedó obsoleto porque el
18+
reset a upstream eliminó los workflows rotos (`python-package.yml`,
19+
`go-build.yml`) que pretendía borrar.
20+
- Diagnósticos de CI hechos por el camino (colisión de símbolos por refactor
21+
paralelo, crash de golangci-lint por Go 1.26 vs golangci 1.25, workflow de
22+
Python sobre repo Go).
23+
24+
## Trabajo vigente sobre el `main` nuevo (upstream)
25+
- **Re-medición de cobertura** desde cero: total **69,3 %**.
26+
- **Tests para `cmd/mcpcurl`**: subido de **8,9 % → ~55 %** añadiendo
27+
`cmd/mcpcurl/coverage_test.go` (sin tocar el test existente). Cubre:
28+
- `buildArgumentsMap` → 100 % (todos los tipos: string/number/integer/boolean/
29+
array de strings/array de objetos vía `-json`, y error de JSON inválido).
30+
- `printResponse` → 88,5 % (sin pretty, objeto, array/JSONL, vacío, JSON inválido).
31+
- `buildJSONRPCRequest` → 75 %.
32+
- `addCommandFromTool` → 63,5 % (registro de flags, flags requeridos, validación
33+
de `enum` vía `PreRunE`).
34+
- Verificado en local: `gofmt`, `go vet ./...`, `go test ./...`**todo verde**.

0 commit comments

Comments
 (0)