docs(links): record C4 hosted verification - #2214
Merged
josecelano merged 4 commits intoSep 14, 2026
Merged
Conversation
There was a problem hiding this comment.
🟡 Changes recommended
One or more issues must be addressed before approval.
Get a fresh assessment by requesting another Copilot review.
Pull request overview
This documentation-only PR records hosted verification for the C4 stale-link repairs after PR #2212 merged.
Changes:
- Updates issue progress and acceptance evidence with run
34843399874. - Records C4 report details and confirms unrelated failures remain visible.
- Appends the independent hosted-verification review.
File summaries
| File | Description |
|---|---|
| docs/issues/open/2185-2003-triage-advisory-external-link-check-findings/ISSUE.md | Updated as part of this pull request. |
| docs/issues/open/2185-2003-triage-advisory-external-link-check-findings/external-link-baseline.md | Updated as part of this pull request. |
| docs/issues/open/2185-2003-triage-advisory-external-link-check-findings/agent-review-reports.md | Updated as part of this pull request. |
Review details
Suppressed comments (3)
docs/issues/open/2185-2003-triage-advisory-external-link-check-findings/ISSUE.md:144
- Use the plural noun here: the sentence refers to the targets for both C3 and C4, not one combined target.
- [ ] AC2: Each repair changes only a verified stale reference and records why its replacement target is correct. C3 and C4 target and hosted verification are recorded; remaining repair slices must be verified before AC2 can be completed.
docs/issues/open/2185-2003-triage-advisory-external-link-check-findings/ISSUE.md:181
- Use the plural noun here: the evidence row refers to the targets for both C3 and C4, not one combined target.
| AC2 | TODO | C3 and C4 target and hosted verification are recorded; remaining repair slices must be verified. |
docs/issues/open/2185-2003-triage-advisory-external-link-check-findings/agent-review-reports.md:114
- This appended review entry still ends in
REVIEW WARNEDand says to add run 34843399874 to AC5, even though this same PR already adds that run atISSUE.md:184. Because the report contract requires corrections to be recorded as a new entry rather than editing history, append a follow-up entry naming this warning and confirming the correction; otherwise the latest review record remains stale for the final diff.
- Correction required: Add run 34843399874 to AC5's evidence row so the documented evidence explicitly includes the hosted rerun for the C4 remediation slice.
- Pending: Keep T2 and AC2 pending for C5-C8 and remaining repair verification. Keep all issue-wide quality, manual-verification, acceptance-review, implementation-completion-review, and closure criteria pending.
- Verdict: REVIEW WARNED.
- Follow-up actions:
- Make the AC5 evidence-row correction.
- Files reviewed: 3/3 changed files
- Comments generated: 2
- Review effort level: Lite
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
josecelano
added a commit
to josecelano/torrust-tracker
that referenced
this pull request
Sep 14, 2026
josecelano
force-pushed
the
2185-record-c4-hosted-verification
branch
from
September 14, 2026 18:45
719b88d to
b93fa5b
Compare
Member
Author
|
ACK b93fa5b |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Records independently reviewed hosted verification for the C4 repository-controlled stale-link repair after PR #2212 merged.
Evidence
developrevision618723d49283432a99ea5604fc31becfa1a63a84.lychee-external-link-reportuploaded successfully and is retained through 2026-09-28.Related to #2185