Skip to content

Remove pullfrog.yml workflow#33

Merged
khaliqgant merged 3 commits into
mainfrom
remove-pullfrog
May 30, 2026
Merged

Remove pullfrog.yml workflow#33
khaliqgant merged 3 commits into
mainfrom
remove-pullfrog

Conversation

@miyaontherelay
Copy link
Copy Markdown
Contributor

Removing the Pullfrog code review bot workflow.

@gemini-code-assist
Copy link
Copy Markdown

Warning

You have reached your daily quota limit. Please wait up to 24 hours and I will start processing your requests again!

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented May 30, 2026

Review Change Stack

📝 Walkthrough

Walkthrough

Removes the .github/workflows/pullfrog.yml workflow, normalizes package.json's files entry to ["dist"], adds completed trajectory artifacts under .trajectories/completed/2026-05/..., and appends "github" to biome.json's files.ignore.

Changes

Maintenance and metadata

Layer / File(s) Summary
Remove pullfrog workflow
.github/workflows/pullfrog.yml
Entire workflow file deleted (57 lines removed). Previously defined a pullfrog job triggered on pull requests that checked out code and executed an agent step with environment secrets.
files array formatting
package.json
files field changed to single-line array literal ["dist"] without altering published contents.
Trajectory completion artifacts
.trajectories/completed/2026-05/traj_cr28ijla8789/summary.md, .trajectories/completed/2026-05/traj_cr28ijla8789/trajectory.json
Adds a trajectory completion summary.md documenting review tasks and a trajectory.json recording execution metadata and a Biome-ignore decision.
Biome ignore update
biome.json
Appends "github" to the files.ignore list.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Poem

🐰 I nibbled lines and set them straight,

Dist packed tidy, workflow's late.
A traj recorded, a ignore made,
The repo hums beneath my shade.
🥕✨

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title directly and clearly summarizes the main change: removal of the pullfrog.yml workflow file.
Description check ✅ Passed The description is directly related to the changeset, accurately describing the removal of the Pullfrog code review bot workflow.
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 remove-pullfrog

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 and usage tips.

Copy link
Copy Markdown

@pullfrog pullfrog Bot left a comment

Choose a reason for hiding this comment

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

✅ No new issues found.

Reviewed changes — this PR removes the Pullfrog code review bot workflow by deleting .github/workflows/pullfrog.yml.

  • Remove pullfrog.yml — deletes the entire workflow (57 lines) which invoked pullfrog/pullfrog@v0 for AI agent code reviews
  • Verified no remaining references to pullfrog secrets or the pullfrog/pullfrog@v0 action elsewhere in the repo; this is a clean, complete removal
  • Other existing workflows (ci.yml, release.yml, compact-on-merge.yml, publish.yml) are unaffected

Pullfrog  | View workflow run | Using Kimi K2𝕏

Copy link
Copy Markdown

@cubic-dev-ai cubic-dev-ai Bot left a comment

Choose a reason for hiding this comment

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

No issues found across 1 file

Re-trigger cubic

Copy link
Copy Markdown

@pullfrog pullfrog Bot left a comment

Choose a reason for hiding this comment

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

✅ No new issues found.

Reviewed changes — this PR removes the Pullfrog code review bot workflow by deleting .github/workflows/pullfrog.yml.

  • Remove pullfrog.yml — deletes the entire workflow (57 lines) which invoked pullfrog/pullfrog@v0 for AI agent code reviews
  • Verified no remaining references to pullfrog secrets or the pullfrog/pullfrog@v0 action elsewhere in the repo; this is a clean, complete removal
  • Other existing workflows (ci.yml, release.yml, compact-on-merge.yml, publish.yml) are unaffected

Pullfrog  | View workflow run | Using Kimi K2𝕏

Copy link
Copy Markdown

@coderabbitai coderabbitai Bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In @.trajectories/completed/2026-05/traj_cr28ijla8789/summary.md:
- Around line 20-22: The document uses the lowercase brand name "github" in
headings and inline text (e.g., the heading "Ignore connector-generated github
mirror in Biome" and the path fragment "github/repos/.../.relay"); update all
occurrences to the proper capitalized form "GitHub" (e.g., "Ignore
connector-generated GitHub mirror in Biome" and "GitHub/repos/.../.relay") to
match branding guidelines.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: b37e2cca-dfe2-4085-a876-973a8fd81f66

📥 Commits

Reviewing files that changed from the base of the PR and between 4200731 and 3d0dfe1.

📒 Files selected for processing (3)
  • .trajectories/completed/2026-05/traj_cr28ijla8789/summary.md
  • .trajectories/completed/2026-05/traj_cr28ijla8789/trajectory.json
  • biome.json
✅ Files skipped from review due to trivial changes (2)
  • biome.json
  • .trajectories/completed/2026-05/traj_cr28ijla8789/trajectory.json

Comment on lines +20 to +22
### Ignore connector-generated github mirror in Biome
- **Chose:** Ignore connector-generated github mirror in Biome
- **Reasoning:** Local PR review tools materialize github/repos/.../.relay JSON under the workspace; npm run lint scans it even though it is not product source or PR content.
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

⚠️ Potential issue | 🟡 Minor | ⚡ Quick win

Capitalize "GitHub" throughout the document.

The platform name should be capitalized as "GitHub" rather than "github" for proper brand representation in documentation.

📝 Proposed fix
-### Ignore connector-generated github mirror in Biome
-- **Chose:** Ignore connector-generated github mirror in Biome
-- **Reasoning:** Local PR review tools materialize github/repos/.../.relay JSON under the workspace; npm run lint scans it even though it is not product source or PR content.
+### Ignore connector-generated GitHub mirror in Biome
+- **Chose:** Ignore connector-generated GitHub mirror in Biome
+- **Reasoning:** Local PR review tools materialize GitHub/repos/.../.relay JSON under the workspace; npm run lint scans it even though it is not product source or PR content.
-- Ignore connector-generated github mirror in Biome: Ignore connector-generated github mirror in Biome
+- Ignore connector-generated GitHub mirror in Biome: Ignore connector-generated GitHub mirror in Biome

As per coding guidelines: static analysis flagged improper capitalization of "GitHub".

Also applies to: 31-31

🧰 Tools
🪛 LanguageTool

[uncategorized] ~22-~22: The official name of this software platform is spelled with a capital “H”.
Context: ...ng:** Local PR review tools materialize github/repos/.../.relay JSON under the workspa...

(GITHUB)

🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In @.trajectories/completed/2026-05/traj_cr28ijla8789/summary.md around lines 20
- 22, The document uses the lowercase brand name "github" in headings and inline
text (e.g., the heading "Ignore connector-generated github mirror in Biome" and
the path fragment "github/repos/.../.relay"); update all occurrences to the
proper capitalized form "GitHub" (e.g., "Ignore connector-generated GitHub
mirror in Biome" and "GitHub/repos/.../.relay") to match branding guidelines.

@khaliqgant khaliqgant merged commit b4a1b41 into main May 30, 2026
8 checks passed
@khaliqgant khaliqgant deleted the remove-pullfrog branch May 30, 2026 17:30
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