Converge connection getter and command helper names across modules
Los mantenedores suelen responder en 1 día
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 5/5
- Tiempo estimado
- Más de una semana
- Aptitud para principiantes
- 48/100
- Tipo de issue
- Refactorización
- Claridad
- Bastante claro
- Estado de actividad
- Activo
- Stack tecnológico
- typescript
- Área
- api
Línea de trabajo
Start by comparing the named getters, credential setters, and command helpers in packages/modules/*/src/*-*.ts, then read the add-module skill and coordinate the old-name handling with #1518. Decide and document shared naming patterns, including protocol-specific URLs and command helpers; done means the affected modules follow those patterns, old names remain deprecated aliases, and the skill records the convention.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
What happens
Modules name the same concept differently, so moving from one module to another means checking the docs each time.
Connection string getters:
| Method | Modules |
|---|---|
getConnectionUri() |
azurecosmosdb, cockroachdb, localstack, mariadb, mssqlserver, mysql, postgresql |
getConnectionString() |
azureservicebus, azurite, hivemq, mongodb, mosquitto |
getConnectionUrl() |
clickhouse, minio, redis, valkey |
getUrl() |
couchdb, influxdb, oraclefree |
getHttpUrl() |
arangodb, chromadb, clickhouse, elasticsearch, opensearch |
getHttpUri(), getBoltUri() |
neo4j |
Other address getters use get*Endpoint (azurecosmosdb, azurite, etcd, gcloud, ollama) and get*Address (qdrant, redpanda, vault, weaviate).
Credential setters: most modules use withUsername and withPassword. mysql and mariadb use withUserPassword and withRootPassword.
Command helpers on started containers:
executeCliCmd(cmd, additionalFlags)in redis and valkeyexecuteQuery(query, additionalFlags, isRoot)in mysqlexecVaultCommands(commands)in vault
Expected
- One name for "the string a client library connects with", used by every module that has one.
- One naming pattern for protocol-specific URLs and for command helpers.
- The old names stay as deprecated aliases.
- The convention is written into the
add-moduleskill so new modules follow it.
Where
packages/modules/*/src/*-container.ts.
Notes
- .NET puts
GetConnectionString()on a sharedIDatabaseContainerinterface that every database module implements. A shared interface here would also let helpers accept "any database container". - Adding aliases is
minor. Removing the old names would bemajor.
Dependencies
- Nothing blocks this. It can start now.
- Coordinate with #1518: for the old names.
- Lenguaje dominante
- TypeScript
- Estrellas
- 2.6k
- Forks
- 274
- Merge medio
- 1 d 14 h
- PR fusionados (30 d)
- 44
Preparar el entorno
Inicia el contenedor de desarrollo del proyecto en tu navegador, con tu propia cuenta de GitHub.
- Incluye un Dockerfile o un archivo de Docker Compose
- Sin plantilla de pull request
- Leer la guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de testcontainers/testcontainers-node
-
enhancement minor
Dificultad 2/5 1-3 horas Aptitud para principiantes 74/100
testcontainers/testcontainers-node#1539 ·
Los mantenedores suelen responder en 1 día
-
enhancement minor
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
testcontainers/testcontainers-node#1533 ·
Los mantenedores suelen responder en 1 día
-
enhancement minor
Dificultad 2/5 1-3 horas Aptitud para principiantes 77/100
testcontainers/testcontainers-node#1531 ·
Los mantenedores suelen responder en 1 día
-
enhancement minor
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
testcontainers/testcontainers-node#1529 ·
Los mantenedores suelen responder en 1 día
-
enhancement minor
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
testcontainers/testcontainers-node#1528 ·
Los mantenedores suelen responder en 1 día
Todos los issues de testcontainers/testcontainers-node
Issues similares
-
bug help wanted
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
Los mantenedores suelen responder en 1 día
-
agent-task risk:low
Dificultad 1/5 1-3 horas Aptitud para principiantes 88/100
guyellis/http-status-check#512 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
microsoft/playwright#43327 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
microsoft/TypeScript#64758 ·
Los mantenedores suelen responder en 1 día
-
enhancement
Dificultad 2/5 1-3 horas Aptitud para principiantes 65/100
opensearch-project/OpenSearch-Dashboards#12937 ·
Los mantenedores suelen responder en 2 días