Skip to content

docs: hygiene open-source — code of conduct, templates d'issue, badges (#T259) - #5

Merged
Axellum merged 2 commits into
mainfrom
docs/oss-hygiene-t259
Aug 10, 2026
Merged

Axellum merged 2 commits into
mainfrom
docs/oss-hygiene-t259

Conversation

@Axellum

@Axellum Axellum commented Aug 10, 2026 •

Copy link
Copy Markdown
Owner

Objectif

Hygiène open-source du miroir public (#T259) : combler les manques signalés dans CONTRIBUTING.md et les README.

Changements

  • CODE_OF_CONDUCT.md — Contributor Covenant v2.1 (EN, texte officiel). E-mail de contact laissé volontairement en placeholder : [INSERT EMAIL ADDRESS] à remplir avant merge (apparaîtra dans Insights → Community Standards).
  • .github/ISSUE_TEMPLATE/bug_report.yml — exige version/commit, plateforme (Windows/Linux/macOS/Docker), étapes de reproduction, extrait de log (avec rappel de masquer les clés API).
  • .github/ISSUE_TEMPLATE/feature_request.yml — problème/motivation, solution proposée, alternatives.
  • README.md — badges statut des 3 workflows CI (ci.yml, python-tests.yml, toolmaker_validate.yml) + licence MIT (badge GitHub standard). Les 4 URLs ont été testées (HTTP 200, image/svg+xml).

Constats (signalés, non corrigés — hors périmètre)

  • core/circuit_breaker.py est cité dans l'arborescence des deux README mais n'existe pas à cet emplacement : le fichier réel est core/llm/circuit_breaker.py (utilisé par core/llm_gateway.py). À corriger dans une autre PR.
  • Le badge « Tests: 60 pytest files » sous-estime la réalité (75 fichiers est_*.py présents) — cosmétique, non bloquant.
  • Tous les autres liens internes des README résolvent (vérifiés un par un).

Vérifications

  • Remplacer [INSERT EMAIL ADDRESS] dans CODE_OF_CONDUCT.md
  • CI verte sur cette branche

Note

Low Risk
Documentation and GitHub metadata only; no application or CI logic changes. CoC placeholder email should be replaced before relying on Community Standards.

Overview
Adds open-source community scaffolding so GitHub Insights and contributors have the expected baseline: a Contributor Covenant v2.1 CODE_OF_CONDUCT.md (enforcement contact is still [INSERT EMAIL ADDRESS] and should be filled before merge), plus structured GitHub issue forms for bugs and feature requests.

The bug template auto-labels bug and requires version/commit, platform, repro steps, log output (with a reminder to redact secrets), and expected behavior; optional config points readers at CONTRIBUTING.md. The feature template labels enhancement and asks for motivation, proposed solution, and alternatives.

README badge row is updated: MIT license uses the standard GitHub license badge, and three CI workflow status badges are added (ci.yml, python-tests.yml, toolmaker_validate.yml).

Reviewed by Cursor Bugbot for commit b7fc462. Configure here.

…s (#T259)

- CODE_OF_CONDUCT.md: Contributor Covenant v2.1 (EN), e-mail de contact en placeholder.
- .github/ISSUE_TEMPLATE/: bug_report.yml + feature_request.yml (GitHub Issue Forms).
- README.md: badges statut des 3 workflows CI + licence MIT (URLs testées).
@Axellum

Axellum commented Aug 10, 2026

Copy link
Copy Markdown
Owner Author

CI : échec pytest préexistant, non lié à cette PR (fichiers docs uniquement).

  • \ModuleNotFoundError: No module named 'mcp.server.fastmcp'\ à la collecte de \ ests/test_mcp_path_traversal.py\ et \ ests/unit/test_delegate_to_gateway.py\ (via \core/mcp_app.py).
  • Cause :
    equirements.txt\ épingle \mcp>=1.27.0\ sans borne haute — une version récente du SDK MCP a déplacé/supprimé \mcp.server.fastmcp. Le dernier run CI sur main (28/07, b04fbcb) était vert : dérive de dépendances, pas une régression du dépôt.
  • Correctif à traiter séparément : épingler la version de \mcp\ (ou migrer vers le package \ astmcp). Le scan anti-fuite bloquant (OSS secrets scan) passe ✅.

…#T259)

mcp 2.0.0 (PyPI, 10/08/2026) supprime le module mcp.server.fastmcp, ce qui casse
l'import de core/mcp_app.py et la collecte pytest sur toute PR.
Dernière 1.x testée OK : mcp 1.29.0 (import FastMCP vérifié).
@Axellum
Axellum merged commit 3e2de77 into main Aug 10, 2026
4 checks passed
@Axellum

Axellum commented Aug 10, 2026

Copy link
Copy Markdown
Owner Author

Résolu — CI verte ✅

Commit \�445bf8\ :
equirements.txt\ épingle désormais \mcp>=1.27.0,<2.0.0\ (mcp 2.0.0, sorti le 10/08, a retiré \mcp.server.fastmcp). Import FastMCP vérifié localement avec mcp 1.29.0. Tous les checks passent (Pytest, test-and-scan, Ruff, OSS secrets scan).

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