Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Original file line number Diff line number Diff line change
Expand Up @@ -5,19 +5,19 @@ status: done
priority: p3
epic: 2003
github-issue: 2185
spec-path: docs/issues/open/2185-2003-triage-advisory-external-link-check-findings/ISSUE.md
spec-path: docs/issues/closed/2185-2003-triage-advisory-external-link-check-findings/ISSUE.md
branch: "2185-2003-triage-advisory-external-link-check-findings"
related-pr: null
last-updated-utc: 2026-09-18 07:10
related-pr: 2258
last-updated-utc: 2026-09-18 09:05
semantic-links:
skill-links:
- create-issue
related-artifacts:
- .github/skills/dev/planning/create-issue/SKILL.md
- docs/issues/open/2003-overhaul-guardrails-and-automation/EPIC.md
- docs/issues/closed/2162-enforce-lychee-and-schedule-external-link-checks/ISSUE.md
- docs/issues/open/2185-2003-triage-advisory-external-link-check-findings/external-link-baseline.md
- docs/issues/open/2185-2003-triage-advisory-external-link-check-findings/agent-review-reports.md
- docs/issues/closed/2185-2003-triage-advisory-external-link-check-findings/external-link-baseline.md
- docs/issues/closed/2185-2003-triage-advisory-external-link-check-findings/agent-review-reports.md
- docs/issues/drafts/refactor-semantic-link-conventions/EPIC.md
- docs/issues/drafts/refactor-semantic-link-conventions/external-link-check-residual-failures-2026-09-18.md
- .github/workflows/external-link-check.yaml
Expand Down Expand Up @@ -123,7 +123,7 @@ A category that needs no repository change is recorded in issue-local evidence w
- [x] Reviewer validated acceptance criteria and updated checkboxes
- [x] Independent reviewer reports recorded in issue-local `agent-review-reports.md` when reviewers received this folder-style specification
- [x] Committer verified spec progress is up to date before commit
- [ ] Issue closed and spec moved from `docs/issues/open/` to `docs/issues/closed/`
- [x] Issue closed and spec moved from `docs/issues/open/` to `docs/issues/closed/`

### Progress Log

Expand Down Expand Up @@ -151,6 +151,7 @@ A category that needs no repository change is recorded in issue-local evidence w
- 2026-09-17 16:44 UTC - Copilot - Added two exact online-only C10 patterns for pull-request `#issuecomment-<id>` and `#pullrequestreview-<id>` anchors to `.github/lychee-online.toml`. A five-link boundary test excluded the three dynamic pull-request anchor forms and the exact C6 issue-comment anchor while retaining `https://github.com/torrust/torrust-tracker/pull/123/files`. Merged as [PR #2255](https://github.com/torrust/torrust-tracker/pull/2255).
- 2026-09-18 06:43 UTC - Copilot - After PR #2255 merged, [run 35315382956](https://github.com/torrust/torrust-tracker/actions/runs/35315382956) ran on merged revision `e6dd8918`: it visibly failed with 7 errors and 4 timeouts, excluded 886 links, and retained a 782-byte `lychee-external-link-report` artifact through 2026-10-02. The report contains no GitHub fragment error of any kind. The residual failures are two Medium `403` responses, one Stack Overflow `403`, four FSF transport failures, and four first-time `https://www.gnu.org/licenses/` timeouts. This hosted-verifies C10 and confirms C7-C8 as persistent, non-stale cases.
- 2026-09-18 07:10 UTC - Copilot - Closure decision with the maintainer: the residual C7-C8 cases and the new GNU timeouts are license boilerplate or background reading that cannot be repaired, and every candidate treatment (replacing citations, excluding hosts, relaxing TLS) is a policy choice outside this task's scope. The verbatim closing report is preserved in `docs/issues/drafts/refactor-semantic-link-conventions/external-link-check-residual-failures-2026-09-18.md`, and the triage insights are recorded in that EPIC draft's "Handoff from Issue #2185" section. No issue-local retrospective is needed because the EPIC handoff is the retrospective. The issue closes when this PR merges.
- 2026-09-18 09:05 UTC - Copilot - PR [#2258](https://github.com/torrust/torrust-tracker/pull/2258) merged into `develop` and closed GitHub issue [#2185](https://github.com/torrust/torrust-tracker/issues/2185) as completed. Moved this issue specification from `docs/issues/open/` to `docs/issues/closed/`.

## Acceptance Criteria

Expand Down
Original file line number Diff line number Diff line change
@@ -1,8 +1,8 @@
---
semantic-links:
related-artifacts:
- docs/issues/open/2185-2003-triage-advisory-external-link-check-findings/ISSUE.md
- docs/issues/open/2185-2003-triage-advisory-external-link-check-findings/external-link-baseline.md
- docs/issues/closed/2185-2003-triage-advisory-external-link-check-findings/ISSUE.md
- docs/issues/closed/2185-2003-triage-advisory-external-link-check-findings/external-link-baseline.md
- docs/issues/drafts/refactor-semantic-link-conventions/EPIC.md
- docs/issues/drafts/refactor-semantic-link-conventions/external-link-check-residual-failures-2026-09-18.md
- .github/lychee-online.toml
Expand Down
4 changes: 2 additions & 2 deletions docs/issues/drafts/refactor-semantic-link-conventions/EPIC.md
Original file line number Diff line number Diff line change
Expand Up @@ -14,8 +14,8 @@ semantic-links:
- docs/issues/closed/2233-2003-tune-unified-pr-review-process/ISSUE.md
- docs/issues/closed/2233-2003-tune-unified-pr-review-process/code-span-path-case-analysis.md
- docs/issues/closed/2233-2003-tune-unified-pr-review-process/code-span-path-case-inventory.tsv
- docs/issues/open/2185-2003-triage-advisory-external-link-check-findings/ISSUE.md
- docs/issues/open/2185-2003-triage-advisory-external-link-check-findings/external-link-baseline.md
- docs/issues/closed/2185-2003-triage-advisory-external-link-check-findings/ISSUE.md
- docs/issues/closed/2185-2003-triage-advisory-external-link-check-findings/external-link-baseline.md
- docs/issues/drafts/refactor-semantic-link-conventions/external-link-check-residual-failures-2026-09-18.md
- .github/lychee-online.toml
- .github/workflows/external-link-check.yaml
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -245,7 +245,7 @@ creation remains subject to maintainer approval of each draft specification.
| 4 | #2158 - Inventory existing Clippy allows | `docs/issues/open/2158-2003-inventory-existing-clippy-allows/ISSUE.md` | Evidence and incremental remediation; does not redesign the linter runner. | #2157 policy conventions, if merged first |
| 5 | #2159 - Adopt folder-style issue specifications | `docs/issues/open/2159-2003-adopt-folder-style-issue-specs/ISSUE.md` | Documentation, templates, and root ADR only; prospective and reversible for new work. | None |
| 6 | #2160 - Persist independent agent review reports | `docs/issues/closed/2160-2003-persist-independent-agent-review-reports/ISSUE.md` | Profile/template documentation and explicit records; does not enforce transitions technically. | #2155 documentation conventions, if merged first |
| 7 | #2185 - Triage advisory external-link check findings | `docs/issues/open/2185-2003-triage-advisory-external-link-check-findings/ISSUE.md` | Evidence-driven refinement of an existing advisory workflow; does not redesign the linter or workflow runner. | #2162 external-link workflow |
| 7 | #2185 - Triage advisory external-link check findings (DONE) | `docs/issues/closed/2185-2003-triage-advisory-external-link-check-findings/ISSUE.md` | Evidence-driven refinement of an existing advisory workflow; does not redesign the linter or workflow runner. | #2162 external-link workflow |
| 8 | #2219 - Unify PR review-processing workflow (DONE) | `docs/issues/closed/2219-2003-unify-pr-review-processing/ISSUE.md` | Low-risk, additive, independently verifiable local/CI formatting parity plus skill, template, and review-audit process improvements; selects no shared automation architecture. | Existing pre-commit, pre-push, and PR-review entry points |
| 9 | #2233 - Tune unified PR-review process (DONE) | `docs/issues/closed/2233-2003-tune-unified-pr-review-process/ISSUE.md` | Evidence-driven follow-up to #2219; refines skills, templates, audit process, and design notes without selecting shared automation architecture. | #2219 PR-review workflow evidence |

Expand Down
Loading