Skip to content

chore: remove orphaned root-level debris#5267

Merged
leaanthony merged 4 commits into
v3-alphafrom
chore/root-debris-cleanup
Apr 29, 2026
Merged

chore: remove orphaned root-level debris#5267
leaanthony merged 4 commits into
v3-alphafrom
chore/root-debris-cleanup

Conversation

@leaanthony

@leaanthony leaanthony commented Apr 29, 2026

Copy link
Copy Markdown
Member

Summary

Drops files and directories that accumulated at the repo root and aren't deployed or referenced anywhere, plus retires three workflows ahead of the v3-alpha → master merge.

Removed:

  • pr_4570.diff — orphaned patch artifact from PR Add webkitgtk6 option #4570
  • test-changelog-extraction.sh, test-ios-compile.sh, test-version-logic.sh — one-off helper scripts; logic that is still needed lives under v3/scripts/
  • test-workflow.md — notes file, no longer current
  • mkdocs-website/ — abandoned MkDocs experiment, not deployed
  • .github/workflows/auto-label-issues.yml — retiring the auto-labeller
  • .github/workflows/test-nightly-releases.yml — unused for 7+ months, superseded by the real nightly/weekly release pipelines
  • .github/workflows/issue-triage-automation.yml — moving to a real agent-driven triage workflow

Companion PR removes the same three workflow files from master, so the merge sees zero conflict on any of them.

Test plan

  • CI green
  • No remaining references: grep -rn "pr_4570.diff\|test-changelog-extraction\|test-ios-compile\|test-version-logic\|test-workflow.md\|mkdocs-website\|auto-label-issues\|test-nightly-releases\|issue-triage-automation" returns no matches.

🤖 Generated with Claude Code

Summary by CodeRabbit

  • Documentation

    • Removed changelog documentation file.
  • Chores

    • Removed automated issue and pull request labeling workflows.
    • Removed internal test scripts and nightly release test suite.

Drops files that accumulated at the repo root and aren't referenced
anywhere:

- pr_4570.diff (orphaned patch artifact from PR #4570)
- test-changelog-extraction.sh, test-ios-compile.sh,
  test-version-logic.sh (one-off helper scripts; logic that is still
  needed lives under v3/scripts/ instead)
- test-workflow.md (notes file, no longer current)

Reduces noise at the repo root and trims surface area before the
v3-alpha → master merge.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Copilot AI review requested due to automatic review settings April 29, 2026 11:31
@coderabbitai

coderabbitai Bot commented Apr 29, 2026

Copy link
Copy Markdown
Contributor

Caution

Review failed

Pull request was closed or merged during review

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 91785aa8-5611-4ffb-8d0e-e5f152458d97

📥 Commits

Reviewing files that changed from the base of the PR and between 430e54b and c670c4e.

📒 Files selected for processing (2)
  • .github/workflows/issue-triage-automation.yml
  • .github/workflows/test-nightly-releases.yml
💤 Files with no reviewable changes (2)
  • .github/workflows/test-nightly-releases.yml
  • .github/workflows/issue-triage-automation.yml

Walkthrough

This pull request deletes multiple test scripts, documentation, and CI workflow files; no new files or replacement logic are added. (Files removed: test scripts, workflow docs, changelog page, and several GitHub Actions workflows.)

Changes

Cohort / File(s) Summary
Test Scripts
test-changelog-extraction.sh, test-ios-compile.sh, test-version-logic.sh
Removed shell test scripts that validated changelog extraction, iOS build workflow, and version/commit-detection logic.
Workflow Documentation
test-workflow.md
Removed manual/nightly workflow testing guide and checklist for release/testing procedures.
Website Changelog
mkdocs-website/docs/en/changelog.md
Removed changelog documentation page content, including unreleased entries and sections.
GitHub Workflows
.github/workflows/auto-label-issues.yml, .github/workflows/issue-triage-automation.yml, .github/workflows/test-nightly-releases.yml
Removed CI workflow files for auto-labeling, issue triage automation, and nightly-release testing (all jobs and steps deleted).

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Suggested reviewers

  • Copilot

Poem

🐰 I nibbled logs and scripts away,

Cleared out workflows in a single day.
No bells, no builds, just tidy ground—
A quiet burrow, neat and sound.
Hooray for crumbs of code I found.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately describes the main change: removing orphaned/unused files and directories from the repository root that are no longer deployed or referenced.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch chore/root-debris-cleanup

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share
Review rate limit: 0/8 reviews remaining, refill in 59 minutes and 49 seconds.

Comment @coderabbitai help to get the list of available commands and usage tips.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Removes orphaned, root-level artifacts (an old patch diff, ad-hoc test scripts, and an outdated workflow notes file) that are not referenced anywhere in the repo.

Changes:

  • Deleted pr_4570.diff (orphaned patch artifact from PR #4570).
  • Deleted one-off helper scripts for changelog extraction, iOS compile testing, and version logic testing.
  • Deleted test-workflow.md (outdated notes about testing the nightly release workflow).

Reviewed changes

Copilot reviewed 4 out of 5 changed files in this pull request and generated no comments.

Show a summary per file
File Description
test-workflow.md Removes outdated workflow testing notes that referenced now-nonexistent local scripts.
test-version-logic.sh Removes an ad-hoc local script; workflow logic appears to live elsewhere now.
test-ios-compile.sh Removes a machine-specific, one-off iOS compile script from repo root.
test-changelog-extraction.sh Removes an ad-hoc changelog extraction script; no remaining references found.
pr_4570.diff Removes an orphaned patch artifact from the repository root.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

leaanthony and others added 3 commits April 29, 2026 21:34
Deletes the entire `mkdocs-website/` directory. It was an
alternative MkDocs-based docs setup, not deployed anywhere
(the v3 docs ship from `docs/` via Cloudflare Pages and
the v2 docs ship from `website/`), and has been confirmed
abandoned.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Companion deletion to the master-side PR; retiring the auto-labeller
rather than keeping it across both versions. Removing here so the
v3-alpha → master merge has no conflict on this file.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Companion deletion to the master-side PR; retiring both ahead of the
v3-alpha → master merge so neither file appears in the conflict set.
Issue triage is moving to a real agent-driven workflow.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@leaanthony leaanthony merged commit d8e7071 into v3-alpha Apr 29, 2026
11 of 13 checks passed
@leaanthony leaanthony deleted the chore/root-debris-cleanup branch April 29, 2026 11:49
@leaanthony leaanthony mentioned this pull request Apr 29, 2026
3 tasks
pull Bot pushed a commit to c3b2a7/wails that referenced this pull request Apr 29, 2026
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