Skip to content

Jier maintanability concern#8

Merged
jier merged 8 commits intomainfrom
jier-maintanability-concern
Mar 9, 2026
Merged

Jier maintanability concern#8
jier merged 8 commits intomainfrom
jier-maintanability-concern

Conversation

@jier
Copy link
Copy Markdown
Owner

@jier jier commented Mar 9, 2026

Housekeeping, quality detection, and release preparation for v0.9.0

What changed

Code quality — unnamed task detection

  • Added a new maintainability check that flags roles with 3+ tasks missing a name: field (INFO severity). Structural modules (include_tasks, import_tasks, block, etc.) are excluded to avoid noise.

  • Fixed two silent regressions where security and duplication detectors used empty-string fallbacks that prevented pattern matching on unnamed tasks entirely. Handlers without name: now emit a warning instead of dropping silently.

Documentation cleanup — 2,088 lines removed

  • Removed five internal reflection/planning documents that leaked into the public repo (ADOPTION_IMPLEMENTATION_GUIDE.md, SMART_PRESETS_CICD_FRIENDLY.md, UX_IMPROVEMENTS_ROADMAP.md, PROJECT_STATUS_SUMMARY.md, FOLDER_REORGANIZATION_REPORT.md). These were working documents not intended for consumers.

CI/CD examples — Azure DevOps added

  • Added examples/ci_pipeline/azure-devops-pipeline.yml (two-stage: validate on all branches, generate docs on main) and updated the CI examples README and project README to include it.

INSTALL.md corrected

  • Fixed broken minimum Python version (3.7 → 3.10), removed non-existent docsible-init-config command, removed Redis optional dependency that was never implemented, replaced non-existent ./example-role paths.
  • Added an Available Commands table covering all six commands including scan collection and suppress.

CACHING_IMPLEMENTATION_GUIDE.md corrected

  • Resolved a self-contradiction where Phase 2 (complexity caching) was marked "Not Yet Implemented" in one section and "implemented" in another. Updated example to use the correct @cache_by_dir_mtime decorator, added collection-level git caching section, and documented the previously undocumented @cache_by_content_hash and cached_resolve_path utilities.

Test script relocated

  • test_orchestrator_cli.sh moved from project root to tests/integration/ via git mv and documented in CONTRIBUTING.md.

Release process

  • scripts/change_version.py removed — superseded by git-based dynamic versioning (docsible/utils/version.py). CONTRIBUTING.md now documents the git-tag release workflow.
  • Version bumped to 0.9.0 across pyproject.toml and constants.py. Tag v0.9.0 created locally, to be pushed from main after merge.

@jier jier merged commit 581415d into main Mar 9, 2026
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