feat(tf-config): implement story 0-1 YAML configuration management#10
Merged
EdouardZemb merged 5 commits intomainfrom Feb 5, 2026
Merged
feat(tf-config): implement story 0-1 YAML configuration management#10EdouardZemb merged 5 commits intomainfrom
EdouardZemb merged 5 commits intomainfrom
Conversation
- Implement config parsing with serde and validation - Add error handling with thiserror - Include integration tests with fixture files - Add project scaffolding (.gitignore, LICENSE, Cargo workspace) - Update Codex tooling and BMAD sprint artifacts Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
- Add extract_field_from_error() for better field identification in serde errors - Enhance string type error handling with specific field detection - Allow numeric folder names (e.g., "2026") as valid output paths - Update tests for numeric path acceptance and error attribution Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
- Update story status to review - Mark all Review 16 items as completed (3/3) - Add completion notes for error parsing and numeric path fixes - Update test count to 147 (136 unit + 8 integration + 3 doc-tests) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
- Add redact_url_userinfo() to mask user:pass@host credentials in URLs - Extend redact_url_sensitive_params() to handle both userinfo and query params - Replace String::leak() with owned String via InvalidEnumValueDynamic variant - Add 7 unit tests for userinfo redaction (154 tests total) - Document Review 17 fixes in story 0-1 Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
- Add traceability matrix mapping requirements to test coverage - Add test review documentation for story 0-1 - Add valid_cloud_config.yaml test fixture - Update tf-config README documentation - Document serde_yaml tech debt in Cargo.toml - Mark story status as done
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Implements Story 0-1: Configurer un projet via config.yaml - adding the
tf-configcrate for centralized YAML configuration management.Changes
Core Implementation
tf-configcrate with YAML configuration parsing and validationconfig.yamlString::leakpatterns)Documentation & QA Artifacts
Testing
valid_cloud_config.yaml)Checklist
Related
Completes Story 0-1 from Sprint 0