Populate empty AGENTS.md sections with RHDH domain context and CLI patterns
Les mainteneurs répondent en général sous 1 jour
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 2/5
- Temps estimé
- 1-3 heures
- Accessibilité débutants
- 78/100
- Type d'issue
- Documentation
- Clarté
- Clairement spécifiée
- Activité
- Active
- Stack technique
- typescript
- Domaine
- cli, developer-experience, documentation
Piste de recherche
Commencez par AGENTS.md, puis lisez les points d’entrée référencés : src/lib/rhdhVersion.ts, src/commands/check-versions/command.ts et src/lib/rhdhVersion.test.ts. Renseignez Key Conventions, Architecture et Pattern References avec le mappage de versions RHDH spécifié, le comportement hors ligne, la structure de la commande, la mise en cache et les modèles de test ; le travail est terminé lorsque les trois sections contiennent un contexte exploitable sur le projet.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
What happened
On PR #176, human reviewer PatAKnight caught 5 design-level concerns about RHDH version resolution semantics, air-gap behavior, and CLI UX that the review agent missed entirely. The agent's successful review ($8.81, opus) produced only code-level findings — valid but lower-impact. The repo's AGENTS.md has empty ## Key Conventions, ## Architecture, and ## Pattern References sections (only HTML comment placeholders), leaving the review agent with no domain context about RHDH releases, Backstage version mapping, the CLI command structure, or air-gapped deployment requirements.
Specific human findings the agent missed:
- Falling back to
mainwhen a release branch doesn't exist could silently resolve the wrong Backstage version RHDH_OFFLINEonly skips GitHub metadata — the Backstage manifest still fetches fromversions.backstage.io, so this is not air-gapped- Unprefixed versions like
1.54.0are silently treated as Backstage versions, which is easy to misread as RHDH - The static compatibility matrix will require manual updates every release cycle
backstage:^peer dependencies should show an explicit unverifiable message rather than silently flagging as invalid
What could go better
The review agent had no way to reason about domain-specific design trade-offs because the AGENTS.md provides only build/test commands and PR conventions — zero architectural context. The human reviewer's design-level comments required understanding: (1) RHDH release branches and their relationship to Backstage versions, (2) the distinction between offline metadata lookup and air-gapped manifest resolution, (3) version input disambiguation (RHDH vs Backstage), and (4) static compatibility matrix maintenance burden.
High confidence that populating these sections would improve agent review depth for feature PRs touching version resolution or CLI commands. Moderate confidence for other PR types — the agent may still miss nuances requiring deep product knowledge, but having architectural context establishes a baseline the agent currently lacks entirely.
Proposed change
Fill in the three empty AGENTS.md sections:
Key Conventions: Document the RHDH-to-Backstage version mapping model (release branches, static matrix fallback, backstage: prefix escape hatch), the offline vs air-gapped distinction (RHDH_OFFLINE skips GitHub metadata but not manifest fetch; --manifest-file is needed for true air-gap), and the convention that CLI commands set process.exitCode rather than calling process.exit() directly for testability.
Architecture: Document the version resolution engine (src/lib/rhdhVersion.ts) as the core abstraction — it resolves RHDH version aliases to Backstage release versions via GitHub metadata with a static matrix fallback. Document the command structure under src/commands/ (each command is a directory with command.ts exporting registration and handler). Note the manifest caching strategy (keyed by version + base URL).
Pattern References: Point to src/commands/check-versions/command.ts as the reference for new CLI commands, src/lib/rhdhVersion.ts for version resolution patterns, and src/lib/rhdhVersion.test.ts for test patterns using mocked fetch responses.
Validation criteria
On the next 3 feature PRs that touch version resolution or add a new CLI command, the review agent should surface at least one design-level finding related to RHDH domain semantics (e.g., version mapping correctness, offline/air-gap behavior, user-facing error messages) rather than only code-level syntax and style findings.
Generated by retro agent from https://github.com/redhat-developer/rhdh-cli/pull/176
- Langage dominant
- TypeScript
- Étoiles
- 6
- Forks
- 15
- Merge moyen
- 1 h
- PR mergées (30 j)
- 1
Préparer son environnement
- Aucun Dockerfile ni fichier Docker Compose
- Aucun modèle de pull request
- Lire le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de redhat-developer/rhdh-cli
-
bug ready-for-triage triaged
Difficulté 3/5 1-2 jours Accessibilité débutants 72/100
redhat-developer/rhdh-cli#203 · 2 commentaires ·
Les mainteneurs répondent en général sous 1 jour
Toutes les issues de redhat-developer/rhdh-cli
Issues similaires
-
area: web bug
Difficulté 2/5 1-3 heures Accessibilité débutants 76/100
kinqsradio/orbyn-engineering#227 ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 85/100
joeldesante/TurfBuilder#258 ·
Les mainteneurs répondent en général sous 1 jour
-
bug
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
ncecere/open-mcp-portal#85 ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 63/100
loathers/garbage-collector#2865 ·
Les mainteneurs répondent en général sous 1 jour
-
test-reliability
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
Azure/azure-sdk-for-js#40672 · 1 réaction ·
Les mainteneurs répondent en général sous 1 jour