Skip to content

chore: resolve nix hashes conflict from PR #10 - #12

Merged
DevFlex-AI merged 5 commits into
devfrom
fix-pr10-conflicts
Jul 30, 2026
Merged

chore: resolve nix hashes conflict from PR #10#12
DevFlex-AI merged 5 commits into
devfrom
fix-pr10-conflicts

Conversation

@DevFlex-AI

@DevFlex-AI DevFlex-AI commented Jul 30, 2026

Copy link
Copy Markdown

Issue for this PR

Closes #13

Type of change

  • Bug fix
  • New feature
  • Refactor / code improvement
  • Documentation

What does this PR do?

Reconciles the divergent chore: update nix node_modules hashes history from #10 so it can be closed. dev was since force-pushed (b991c17) to adopt upstream history that already contains #10's hash commit (3f239b3), so the latest merge resolves the nix/hashes.json conflict by keeping dev's current values; the file is byte-identical to dev and the merge introduces no hash changes.

It also restores the Auto Assign workflow (dropped from dev by the force-push) upgraded from pozil/auto-assign-issue@v1 to v2, pinned to the v2.2.0 commit SHA: v1 does not support pull_request events (it throws "Couldn't find issue info in current context") and rejects the numOfAssignee input, so the workflow failed on every PR.

How did you verify your code works?

Confirmed nix/hashes.json on this branch is byte-identical to dev, so the merge is content-neutral. The auto-assign fix matches the v2 action's documented inputs (repo-token, assignees, numOfAssignee), which v2 supports for both issues and PRs.

Screenshots / recordings

Not a UI change.

Checklist

  • I have tested my changes locally
  • I have not included unrelated changes in this PR

View with [code]smith
Need help on this PR? Tag @codesmith-bot with what you need. Autofix is enabled.

# Conflicts:
#	nix/hashes.json

Co-authored-by: Codesmith <codesmith-bot@users.noreply.github.com>
@vercel

vercel Bot commented Jul 30, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
bolt-cli-app Ready Ready Preview Jul 30, 2026 5:48pm

@github-actions

Copy link
Copy Markdown

Thanks for your contribution!

This PR doesn't have a linked issue. All PRs must reference an existing issue.

Please:

  1. Open an issue describing the bug/feature (if one doesn't exist)
  2. Add Fixes #<number> or Closes #<number> to this PR description

See CONTRIBUTING.md for details.

@github-actions

Copy link
Copy Markdown

This PR doesn't fully meet our contributing guidelines and PR template.

What needs to be fixed:

  • PR description is missing required template sections. Please use the PR template.

Please edit this PR description to address the above within 2 hours, or it will be automatically closed.

If you believe this was flagged incorrectly, please let a maintainer know.

@github-actions

Copy link
Copy Markdown

The following comment was made by an LLM, it may be inaccurate:

Co-authored-by: Codesmith <codesmith-bot@users.noreply.github.com>
@coderabbitai

coderabbitai Bot commented Jul 30, 2026

Copy link
Copy Markdown

Review Change Stack

📝 Walkthrough

Walkthrough

The auto-assign workflow action reference was updated from v1 to v2; all other workflow configuration remains unchanged.

Changes

Auto-assign workflow upgrade

Layer / File(s) Summary
Update auto-assign action version
.github/workflows/auto-assign.yml
The workflow now uses pozil/auto-assign-issue@v2 instead of v1, with existing wiring and inputs preserved.

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

Possibly related issues

  • bolt-builder/bolt-cli issue 13 — Directly tracks updating pozil/auto-assign-issue from v1 to v2 for pull request event support.

Poem

A bunny checks the workflow line,
“v2,” it says, “the hops align!”
The triggers stay, the inputs too,
One tiny change makes all things new.
Nibble nibble—merge it through!

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
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.
Title check ✅ Passed The title is clearly related to the nix hashes conflict resolution, though it omits the workflow restore and is slightly narrower than the full change.
Description check ✅ Passed The description matches the template and covers the issue, change type, implementation details, verification, screenshots, and checklist.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix-pr10-conflicts

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

@github-actions

Copy link
Copy Markdown

Thanks for updating your PR! It now meets our contributing guidelines. 👍

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown

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 @.github/workflows/auto-assign.yml:
- Line 15: Update the action reference in the workflow step using
pozil/auto-assign-issue so it is pinned to a full commit SHA instead of the
mutable v2 tag, while retaining v2 in an adjacent comment for readability.
🪄 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: ASSERTIVE

Plan: Pro Plus

Run ID: 183fc1b3-43e9-413b-b213-e4ce2d153f8b

📥 Commits

Reviewing files that changed from the base of the PR and between 2bef5ce and fb386c7.

📒 Files selected for processing (1)
  • .github/workflows/auto-assign.yml

Comment thread .github/workflows/auto-assign.yml Outdated
Co-authored-by: Codesmith <codesmith-bot@users.noreply.github.com>
DevFlex-AI and others added 2 commits July 30, 2026 17:45
Co-authored-by: Codesmith <codesmith-bot@users.noreply.github.com>
@DevFlex-AI
DevFlex-AI merged commit 9909bc4 into dev Jul 30, 2026
6 of 12 checks passed
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.

nix/hashes.json conflict between PR #10 and dev

1 participant