docs: reorganize implementation plan and expand contributing guidelines #409

Merged
freemo merged 17 commits from chore/removed-imp into master 2026-02-24 03:22:32 +00:00
Owner

Summary

This PR reorganizes the project's documentation by breaking up the monolithic
implementation_plan.md and substantially expanding CONTRIBUTING.md with
process guidelines that were previously missing or buried in the plan.

Changes

  • Migrated implementation timeline from implementation_plan.md to docs/timeline.md.
  • Migrated implementation notes to docs/implementation-notes.md.
  • Relocated remaining content from the implementation plan to the specification and
    CONTRIBUTING.md, then deleted implementation_plan.md from the repository.
  • Expanded CONTRIBUTING.md with comprehensive guidance on:
    • Commit scope, quality, and message format (Conventional Changelog).
    • Issue creation process, including required sections (Metadata, Acceptance Criteria,
      Definition of Done).
    • Label system (State/, Priority/, MoSCoW/, Type/, special labels).
    • Ticket lifecycle and state transition rules.
    • Pull Request process, review requirements, and merge checklist.
    • Milestone requirements for issues and PRs.
    • PR labeling rules (Type/ labels on PRs).
    • PR description requirements (detailed body, closing keywords, dependency links).
    • Project-specific conventions (tooling, testing, type checking, code style).
  • Updated CHANGELOG.md with entries for all documentation changes.

Motivation

Contributors and automated agents lacked clear, centralized documentation on
how to create issues, write commit messages, submit PRs, and follow the
project's label and lifecycle conventions. The implementation plan contained
useful information but was a monolithic file mixing timelines, notes, and
process guidelines. This PR gives each concern its own home and makes the
contributing guidelines self-contained.

Closes #408

## Summary This PR reorganizes the project's documentation by breaking up the monolithic `implementation_plan.md` and substantially expanding `CONTRIBUTING.md` with process guidelines that were previously missing or buried in the plan. ### Changes - **Migrated implementation timeline** from `implementation_plan.md` to `docs/timeline.md`. - **Migrated implementation notes** to `docs/implementation-notes.md`. - **Relocated remaining content** from the implementation plan to the specification and CONTRIBUTING.md, then **deleted `implementation_plan.md`** from the repository. - **Expanded CONTRIBUTING.md** with comprehensive guidance on: - Commit scope, quality, and message format (Conventional Changelog). - Issue creation process, including required sections (Metadata, Acceptance Criteria, Definition of Done). - Label system (`State/`, `Priority/`, `MoSCoW/`, `Type/`, special labels). - Ticket lifecycle and state transition rules. - Pull Request process, review requirements, and merge checklist. - Milestone requirements for issues and PRs. - PR labeling rules (`Type/` labels on PRs). - PR description requirements (detailed body, closing keywords, dependency links). - Project-specific conventions (tooling, testing, type checking, code style). - **Updated CHANGELOG.md** with entries for all documentation changes. ### Motivation Contributors and automated agents lacked clear, centralized documentation on how to create issues, write commit messages, submit PRs, and follow the project's label and lifecycle conventions. The implementation plan contained useful information but was a monolithic file mixing timelines, notes, and process guidelines. This PR gives each concern its own home and makes the contributing guidelines self-contained. Closes #408
Refs: #408
Refs: #408
Refs: #408
Refs: #408
Refs: #408
Refs: #408
Refs: #408
Refs: #408
docs: update changelog with documentation reorganization entries
Some checks failed
CI / benchmark-publish (pull_request) Has been skipped
CI / lint (pull_request) Successful in 14s
CI / build (pull_request) Successful in 17s
CI / quality (pull_request) Successful in 20s
CI / security (pull_request) Successful in 29s
CI / typecheck (pull_request) Successful in 34s
CI / integration_tests (pull_request) Successful in 3m46s
CI / unit_tests (pull_request) Successful in 7m13s
CI / docker (pull_request) Successful in 39s
CI / benchmark-regression (pull_request) Successful in 20m19s
CI / coverage (pull_request) Has been cancelled
ccff33c7c7
Added an Unreleased section to CHANGELOG.md describing all documentation
changes in this branch: CONTRIBUTING.md expansion, implementation plan
migration to docs/timeline.md and docs/implementation-notes.md, and
removal of the monolithic implementation_plan.md.

Refs: #408
freemo added this to the v3.0.0 milestone 2026-02-24 00:51:40 +00:00
docs: add milestone and PR label requirements to contributing guidelines
Some checks failed
CI / benchmark-publish (pull_request) Has been skipped
CI / lint (pull_request) Successful in 14s
CI / build (pull_request) Successful in 14s
CI / quality (pull_request) Successful in 18s
CI / typecheck (pull_request) Successful in 31s
CI / security (pull_request) Successful in 35s
CI / integration_tests (pull_request) Successful in 2m45s
CI / unit_tests (pull_request) Successful in 6m53s
CI / docker (pull_request) Has been cancelled
CI / coverage (pull_request) Has been cancelled
CI / benchmark-regression (pull_request) Has been cancelled
31094b20b9
Added milestone requirements: issues beyond State/Unverified must be assigned
to a milestone; PRs must be assigned to the same milestone as their linked
issues. Added PR label rules: every PR must carry exactly one Type/ label
matching the nature of the change. Updated the triaging checklist to include
milestone assignment and updated the merge checklist with milestone and label
verification items.

Refs: #408
docs: add PR description and dependency link requirements to contributing guidelines
Some checks failed
CI / benchmark-publish (pull_request) Has been skipped
CI / lint (pull_request) Successful in 15s
CI / quality (pull_request) Successful in 16s
CI / build (pull_request) Successful in 22s
CI / security (pull_request) Successful in 29s
CI / typecheck (pull_request) Successful in 30s
CI / integration_tests (pull_request) Successful in 4m3s
CI / coverage (pull_request) Has been cancelled
CI / benchmark-regression (pull_request) Has been cancelled
CI / unit_tests (pull_request) Has been cancelled
CI / docker (pull_request) Has been cancelled
bdda79f24b
Added explicit requirement that every PR must include a detailed description
with a summary, closing keywords for linked issues, and a Forgejo dependency
link. Updated the merge checklist with PR description and dependency link
verification items. Reworked PR Process item 1 to be comprehensive about
both the description body and the issue linkage.

Refs: #408
docs: require branch/tag ref field on all issues in contributing guidelines
Some checks failed
CI / benchmark-publish (pull_request) Has been skipped
CI / lint (pull_request) Successful in 15s
CI / build (pull_request) Successful in 16s
CI / quality (pull_request) Successful in 19s
CI / typecheck (pull_request) Successful in 31s
CI / security (pull_request) Successful in 37s
CI / integration_tests (pull_request) Successful in 3m50s
CI / benchmark-regression (pull_request) Has been cancelled
CI / coverage (pull_request) Has been cancelled
CI / unit_tests (pull_request) Has been cancelled
CI / docker (pull_request) Has been cancelled
0d5e97ff62
Added requirement that every issue must have its Forgejo branch/tag (Ref) field
set to the working branch once work begins. This field is separate from the
branch name in the issue body Metadata section but must match it. Setting the
field allows Forgejo to associate issues with branches in its UI.

Refs: #408
docs: clarify branch/tag ref rules for development vs non-development issues
Some checks failed
CI / benchmark-publish (pull_request) Has been skipped
CI / lint (pull_request) Successful in 16s
CI / build (pull_request) Successful in 15s
CI / quality (pull_request) Successful in 19s
CI / security (pull_request) Successful in 30s
CI / typecheck (pull_request) Successful in 42s
CI / integration_tests (pull_request) Successful in 2m42s
CI / unit_tests (pull_request) Successful in 6m44s
CI / docker (pull_request) Successful in 21s
CI / coverage (pull_request) Has been cancelled
CI / benchmark-regression (pull_request) Has been cancelled
a7c51ed78c
Refined the branch/tag (Ref) field requirement: development issues (features,
fixes, chores) must set it to the same branch as the one in the Metadata
section once the issue reaches State/In Progress, and must always reference a
branch not a tag. Non-development issues (bug reports, questions, support)
may leave it blank or optionally point to the branch or tag being referenced.

Refs: #408
freemo self-assigned this 2026-02-24 01:13:19 +00:00
docs: add missing reference pages to mkdocs navigation
Some checks failed
CI / benchmark-publish (pull_request) Has been skipped
CI / lint (pull_request) Successful in 14s
CI / build (pull_request) Successful in 16s
CI / quality (pull_request) Successful in 19s
CI / typecheck (pull_request) Successful in 31s
CI / security (pull_request) Successful in 32s
CI / unit_tests (pull_request) Has been cancelled
CI / coverage (pull_request) Has been cancelled
CI / benchmark-regression (pull_request) Has been cancelled
CI / integration_tests (pull_request) Has been cancelled
CI / docker (pull_request) Has been cancelled
d9a492c3ff
Added decision_model, resource_handlers, and resources to the manually-authored
reference page nav entries in gen_ref_pages.py so mkdocs no longer warns about
pages existing in docs/reference/ but not included in the nav.

Refs: #408
freemo force-pushed chore/removed-imp from 625592d4d1
Some checks failed
CI / benchmark-publish (pull_request) Has been skipped
CI / lint (pull_request) Successful in 15s
CI / quality (pull_request) Successful in 20s
CI / build (pull_request) Successful in 26s
CI / security (pull_request) Successful in 32s
CI / typecheck (pull_request) Successful in 39s
CI / benchmark-regression (pull_request) Has been cancelled
CI / unit_tests (pull_request) Has been cancelled
CI / integration_tests (pull_request) Has been cancelled
CI / coverage (pull_request) Has been cancelled
CI / docker (pull_request) Has been cancelled
to b1bf2250fb
All checks were successful
CI / benchmark-publish (pull_request) Has been skipped
CI / lint (pull_request) Successful in 14s
CI / build (pull_request) Successful in 18s
CI / quality (pull_request) Successful in 20s
CI / security (pull_request) Successful in 29s
CI / typecheck (pull_request) Successful in 34s
CI / integration_tests (pull_request) Successful in 3m44s
CI / unit_tests (pull_request) Successful in 7m9s
CI / docker (pull_request) Successful in 15s
CI / benchmark-regression (pull_request) Successful in 20m19s
CI / coverage (pull_request) Successful in 23m47s
2026-02-24 01:17:41 +00:00
Compare
Docs: Removed implementation-notes and migrated that information as comments on tickets
Some checks failed
CI / benchmark-publish (pull_request) Has been skipped
CI / lint (pull_request) Successful in 13s
CI / build (pull_request) Successful in 15s
CI / quality (pull_request) Successful in 19s
CI / typecheck (pull_request) Successful in 32s
CI / security (pull_request) Successful in 53s
CI / integration_tests (pull_request) Successful in 3m29s
CI / unit_tests (pull_request) Successful in 6m46s
CI / docker (pull_request) Successful in 39s
CI / benchmark-regression (pull_request) Has been cancelled
CI / coverage (pull_request) Has been cancelled
67b4ddf5f8
Docs: Explicitly indicated dependency direction for blocking characteristics in CONTRIBUTING.md
All checks were successful
CI / benchmark-publish (pull_request) Has been skipped
CI / lint (pull_request) Successful in 15s
CI / quality (pull_request) Successful in 18s
CI / build (pull_request) Successful in 20s
CI / security (pull_request) Successful in 29s
CI / typecheck (pull_request) Successful in 32s
CI / integration_tests (pull_request) Successful in 3m12s
CI / unit_tests (pull_request) Successful in 6m51s
CI / docker (pull_request) Successful in 15s
CI / benchmark-regression (pull_request) Successful in 16m15s
CI / coverage (pull_request) Successful in 23m13s
CI / lint (push) Successful in 13s
CI / quality (push) Successful in 18s
CI / build (push) Successful in 21s
CI / typecheck (push) Successful in 33s
CI / benchmark-regression (push) Has been skipped
CI / security (push) Successful in 36s
CI / integration_tests (push) Successful in 2m33s
CI / unit_tests (push) Successful in 6m58s
CI / docker (push) Successful in 8s
CI / benchmark-publish (push) Successful in 10m17s
CI / coverage (push) Successful in 22m57s
417d17478f
freemo scheduled this pull request to auto merge when all checks succeed 2026-02-24 02:58:56 +00:00
freemo merged commit 417d17478f into master 2026-02-24 03:22:32 +00:00
freemo deleted branch chore/removed-imp 2026-02-24 03:22:32 +00:00
Sign in to join this conversation.
No reviewers
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Reference
cleveragents/cleveragents-core!409
No description provided.