Skip to content

Remove all stubs- files from zips - #3272

Merged
Crabcyborg merged 1 commit into
masterfrom
remove_add_stubs-_files_from_zips
Aug 24, 2026
Merged

Remove all stubs- files from zips#3272
Crabcyborg merged 1 commit into
masterfrom
remove_add_stubs-_files_from_zips

Conversation

@Crabcyborg

@Crabcyborg Crabcyborg commented Aug 24, 2026

Copy link
Copy Markdown
Contributor

Summary by CodeRabbit

  • Chores
    • Updated archive packaging rules to exclude all stub directories consistently.

@coderabbitai

coderabbitai Bot commented Aug 24, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro Plus

Run ID: 69ae9844-7b96-45c0-a776-bcb430105d11

📥 Commits

Reviewing files that changed from the base of the PR and between faa3000 and b70d122.

📒 Files selected for processing (1)
  • bin/zip-plugin.sh

Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review.


📝 Walkthrough

Walkthrough

The plugin archive command now excludes all paths that match */stubs-*, replacing two specific stub directory exclusions.

Changes

Plugin archive exclusions

Layer / File(s) Summary
Broaden stub exclusions
bin/zip-plugin.sh
The zip command excludes all stubs-* paths instead of only stubs-mcp-adapter and stubs-wp-rest-controller.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Merge Risk: ⚪ Minimal · up to b70d1

This localized change removes stub files from generated ZIPs, and no actionable merge-blocking risk remains beyond normal checks and review.

🚥 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 clearly and concisely describes the main change: excluding all files matching the stubs-* pattern from ZIP archives.
Docstring Coverage ✅ Passed Docstring check was indeterminate for this PR — some files could not be analyzed in time. Not blocking.
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.
✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch remove_add_stubs-_files_from_zips

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

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

@deepsource-io

deepsource-io Bot commented Aug 24, 2026

Copy link
Copy Markdown

DeepSource Code Review

We reviewed changes in faa3000...b70d122 on this pull request. Below is the summary for the review, and you can see the individual issues we found as inline review comments.

See full review on DeepSource ↗

PR Report Card

Overall Grade   Security  

Reliability  

Complexity  

Hygiene  

Code Review Summary

Analyzer Status Updated (UTC) Details
PHP Aug 24, 2026 2:19p.m. Review ↗
JavaScript Aug 24, 2026 2:19p.m. Review ↗

Important

AI Review is run only on demand for your team. We're only showing results of static analysis review right now. To trigger AI Review, comment @deepsourcebot review on this thread.

@Crabcyborg
Crabcyborg merged commit 59b426d into master Aug 24, 2026
27 of 37 checks passed
@Crabcyborg
Crabcyborg deleted the remove_add_stubs-_files_from_zips branch August 24, 2026 14:23
@Crabcyborg Crabcyborg removed this from the 6.35 milestone Aug 24, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant