Skip to content

chore(issues): archive eight completed issue specs - #2058

Merged
josecelano merged 4 commits into
torrust:developfrom
josecelano:chore/cleanup-completed-issues
Aug 17, 2026
Merged

chore(issues): archive eight completed issue specs#2058
josecelano merged 4 commits into
torrust:developfrom
josecelano:chore/cleanup-completed-issues

Conversation

@josecelano

Copy link
Copy Markdown
Member

Summary

Archives the completed specifications for #889, #1136, #1415, #1453, #2019, #2022, #2036, and #2041 from docs/issues/open/ to docs/issues/closed/.

  • Verified every archived GitHub issue is CLOSED and its implementation PR is merged.
  • Updated archived frontmatter, workflow checkpoints, progress logs, and issue-local evidence metadata.
  • Repaired live references in EPICs, active issue specs, and ADRs while retaining historical review records.
  • Updated the completed-issue cleanup skill to require link auditing and frontmatter maintenance for affected documents.

Validation

  • ./contrib/dev-tools/git/hooks/pre-commit.sh
  • Push-triggered pre-push checks: nightly formatting, workspace checks, documentation build, and stable test suite.

@josecelano
josecelano requested a review from a team as a code owner August 17, 2026 08:56
Copilot AI lite review requested due to automatic review settings August 17, 2026 08:56
@codecov

codecov Bot commented Aug 17, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 82.06%. Comparing base (2b84ac6) to head (27f2ff4).

Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #2058      +/-   ##
===========================================
- Coverage    82.11%   82.06%   -0.05%     
===========================================
  Files          349      349              
  Lines        24845    24845              
  Branches     24845    24845              
===========================================
- Hits         20401    20389      -12     
- Misses        4139     4150      +11     
- Partials       305      306       +1     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@josecelano josecelano self-assigned this Aug 17, 2026

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR archives eight completed issue specifications from docs/issues/open/ to docs/issues/closed/, and repairs live documentation references so current EPICs/ADRs/specs continue to resolve after the move.

Changes:

  • Moved the completed issue specs (#889, #1136, #1415, #1453, #2019, #2022, #2036, #2041) into docs/issues/closed/ and updated their frontmatter (status, spec-path, last-updated-utc) plus related evidence metadata.
  • Updated active EPICs, open issue specs, and ADRs to point at the new docs/issues/closed/... locations.
  • Expanded the cleanup-completed-issues skill and docs/issues/closed/README.md to codify link-auditing + frontmatter maintenance during archiving.

Reviewed changes

Copilot reviewed 21 out of 23 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
docs/issues/open/2039-normalize-per-instance-event-metrics-policy/ISSUE.md Updates metadata and semantic links to reference archived #2036 spec.
docs/issues/open/2035-fix-duplicate-port-zero-tracker-instance-bootstrap/ISSUE.md Repairs references to archived #2036/#2041 specs and updates last-updated metadata.
docs/issues/open/2003-overhaul-guardrails-and-automation/EPIC.md Updates link to the archived #2019 spec.
docs/issues/open/1978-configuration-overhaul-epic/EPIC.md Updates subissue table + links to archived specs (#1136, #1415, #1453, #889).
docs/issues/open/1419-allow-multiple-integration-tests-at-main-app-level/ISSUE.md Repairs prerequisite link to archived #2036.
docs/issues/open/1419-allow-multiple-integration-tests-at-main-app-level/investigation-registar-and-health-check.md Repairs link to archived #2036.
docs/issues/closed/README.md Adds archive-maintenance guidance and points to the authoritative cleanup skill.
docs/issues/closed/889-1978-new-config-option-for-logging-style.md Archives #889 spec (frontmatter + progress log + workflow checkpoint update).
docs/issues/closed/2041-migrate-runtime-service-registry-metadata/ISSUE.md Archives #2041 spec (frontmatter + workflow checkpoint updates + progress log).
docs/issues/closed/2041-migrate-runtime-service-registry-metadata/evidence.md Adds frontmatter for archived evidence file and links back to the parent spec.
docs/issues/closed/2036-add-runtime-service-registry-metadata/ISSUE.md Archives #2036 spec (frontmatter + progress log / checklist alignment).
docs/issues/closed/2022-vendor-and-document-maintainer-merge-workflow/ISSUE.md Archives #2022 spec and updates artifact links to the archived folder.
docs/issues/closed/2022-vendor-and-document-maintainer-merge-workflow/github-merge.py Moves the vendored merge tool into the archived spec folder.
docs/issues/closed/2022-vendor-and-document-maintainer-merge-workflow/COPYING Moves the MIT license text into the archived spec folder.
docs/issues/closed/2019-automatically-format-project-dictionary/ISSUE.md Archives #2019 spec (frontmatter + progress log / checklist update).
docs/issues/closed/1453-1978-ip-bans-reset-interval-configurable/ISSUE.md Archives #1453 spec and updates evidence folder path references.
docs/issues/closed/1453-1978-ip-bans-reset-interval-configurable/evidence/2026-07-24-manual-runtime-verification.md Adds/updates frontmatter to reflect archived location and correct related-artifact links.
docs/issues/closed/1415-1978-use-service-binding-instead-of-socket-addr/ISSUE.md Archives #1415 spec (frontmatter + workflow checkpoint updates).
docs/issues/closed/1415-1978-use-service-binding-instead-of-socket-addr/manual-verification.md Adds frontmatter linking manual verification evidence to the archived spec.
docs/issues/closed/1136-1978-configurable-udp-connection-id-validation-policy.md Archives #1136 spec (frontmatter + progress log / checklist update).
docs/adrs/20260728115400_define_registar_as_runtime_service_registry.md Repairs ADR frontmatter artifact link to archived #2036 spec.
docs/adrs/20260727000000_events_are_objective_facts.md Repairs ADR frontmatter artifact link to archived #1136 spec.
.github/skills/dev/planning/cleanup-completed-issues/SKILL.md Strengthens the archiving workflow: mandatory link-audit step + frontmatter maintenance guidance.
Suppressed comments (4)

docs/issues/closed/889-1978-new-config-option-for-logging-style.md:145

  • This spec is marked status: done and has all acceptance criteria checked, but the workflow checkpoint "Acceptance criteria reviewed after implementation" is still unchecked. That makes the progress tracking inconsistent for an archived/closed issue.
    docs/issues/closed/2041-migrate-runtime-service-registry-metadata/ISSUE.md:254
  • This issue spec is archived and marked status: done, but the workflow checkpoint "Acceptance criteria reviewed after implementation" is still unchecked, which leaves the progress state ambiguous after closure.
    docs/issues/closed/2022-vendor-and-document-maintainer-merge-workflow/ISSUE.md:93
  • This spec is archived as done, but several key workflow checkpoints in this section remain unchecked (e.g., manual verification / acceptance-criteria review / committer verification), and the Acceptance Criteria section below is still entirely unchecked. That makes the archived spec read as incomplete even though the progress log says the implementation PR was merged; the checkboxes should be updated to reflect the actual completion state (or explicitly marked as deferred/N/A).
    docs/issues/closed/1136-1978-configurable-udp-connection-id-validation-policy.md:233
  • This spec is marked status: done and archived, but the workflow checkpoints here still show manual verification / acceptance-criteria review / reviewer validation as incomplete, and the Acceptance Criteria section further down remains unchecked. For a closed/archived spec, either update these checkboxes to reflect the actual completed evidence (or annotate explicitly what is deferred and where).

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread .github/skills/dev/planning/cleanup-completed-issues/SKILL.md Outdated
@josecelano

Copy link
Copy Markdown
Member Author

ACK aae61be

@josecelano
josecelano merged commit 040d939 into torrust:develop Aug 17, 2026
30 of 31 checks passed
@josecelano
josecelano deleted the chore/cleanup-completed-issues branch August 17, 2026 15:48
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.

2 participants