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
freemo added 9 commits 2026-02-24 00:30:59 +00:00
Refs: #408
Refs: #408
Refs: #408
Refs: #408
Refs: #408
Refs: #408
Refs: #408
Refs: #408
docs: update changelog with documentation reorganization entries
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
freemo added the
Type
Task
label 2026-02-24 00:51:40 +00:00
freemo added 1 commit 2026-02-24 00:52:01 +00:00
docs: add milestone and PR label requirements to contributing guidelines
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
freemo added 1 commit 2026-02-24 00:59:14 +00:00
docs: add PR description and dependency link requirements to contributing guidelines
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
freemo added 1 commit 2026-02-24 01:03:54 +00:00
docs: require branch/tag ref field on all issues in contributing guidelines
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
freemo added 1 commit 2026-02-24 01:07:59 +00:00
docs: clarify branch/tag ref rules for development vs non-development issues
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
freemo added 1 commit 2026-02-24 01:15:32 +00:00
docs: add missing reference pages to mkdocs navigation
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 to b1bf2250fb 2026-02-24 01:17:41 +00:00 Compare
freemo added 1 commit 2026-02-24 02:48:14 +00:00
Docs: Removed implementation-notes and migrated that information as comments on tickets
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
freemo added 1 commit 2026-02-24 02:58:42 +00:00
Docs: Explicitly indicated dependency direction for blocking characteristics in CONTRIBUTING.md
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
freemo added the
State
Completed
label 2026-03-04 00:59:09 +00:00
Sign in to join this conversation.
No Reviewers
1 Participants
Notifications
Due Date
No due date set.
Reference: cleveragents/cleveragents-core#409