Skip to content

Update dependicus comment#90

Open
anjoola wants to merge 1 commit into
mainfrom
ag/dependicus-comment
Open

Update dependicus comment#90
anjoola wants to merge 1 commit into
mainfrom
ag/dependicus-comment

Conversation

@anjoola
Copy link
Copy Markdown
Contributor

@anjoola anjoola commented Jun 2, 2026

Note

Low Risk
No logic changes in GitHub issue search; lockfile format change may affect install reproducibility if Yarn versions differ across environments.

Overview
This PR rewords the JSDoc on searchDependicusIssues in GitHubIssueService so draft vs ready-for-review behavior is described in plainer language (no “bot that honks” wording). Runtime behavior is unchanged—draft items are still skipped via issue.draft, and non-draft PRs still count toward open Dependicus totals.

The yarn.lock was regenerated from Yarn Berry’s lockfile format (__metadata, checksums) to classic Yarn v1 (# yarn lockfile v1 with resolved / integrity entries). That is a tooling/format shift, not an application feature change; verify CI and local installs still use the intended Yarn major version.

The PR title/description are very short; if they are meant to match the repo template, consider refreshing metadata (e.g. /fix-pr-metadata) so reviewers see scope and test notes up front.

Reviewed by Cursor Bugbot for commit 63e5f5a. Bugbot is set up for automated code reviews on this repo. Configure here.

@anjoola anjoola requested review from irskep and mblair June 2, 2026 22:39
@anjoola anjoola self-assigned this Jun 2, 2026
Copy link
Copy Markdown

@cursor cursor Bot left a comment

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes using default effort and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit c0911dd. Configure here.

Comment thread yarn.lock Outdated
@anjoola anjoola force-pushed the ag/dependicus-comment branch 2 times, most recently from 63e5f5a to d40d1d6 Compare June 2, 2026 23:17
@anjoola
Copy link
Copy Markdown
Contributor Author

anjoola commented Jun 2, 2026

@irskep I'm not really sure how to fix the failing lockfile check here...

@irskep
Copy link
Copy Markdown
Collaborator

irskep commented Jun 3, 2026

@anjoola what does mise update-all-lockfiles do?

@anjoola
Copy link
Copy Markdown
Contributor Author

anjoola commented Jun 3, 2026

@anjoola what does mise update-all-lockfiles do?

I can't run it fully

184 packages installed [367.00ms]
Done. node_modules installed by bun.
=== Switching to aube ===
Switching to aube...
mise ERROR "aube" couldn't exec process: No such file or directory
mise ERROR Run with --verbose or MISE_VERBOSE=1 for more information

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants