Skip to content

Conversation

@devin-ai-integration
Copy link
Contributor

@devin-ai-integration devin-ai-integration bot commented Nov 23, 2025

Add dasel installation to setup-devtools.sh

Summary

Added dasel (a JSON/YAML/TOML/XML query tool) installation to the developer tools setup script. The installation follows the same conditional pattern as other tools in the script - it checks if dasel is already installed before attempting to download and install it.

The installation command downloads the latest linux_amd64 binary from the dasel GitHub releases, makes it executable, and moves it to /usr/local/bin/.

Review & Testing Checklist for Human

  • Test the installation on a clean Linux system - The nested curl command to fetch the download URL from GitHub API is complex and untested. Verify it correctly extracts the URL and downloads the right binary.
  • Check GitHub API rate limiting behavior - The script makes an unauthenticated API call to GitHub. Consider what happens if rate limits are hit or the API response format changes.
  • Verify script behavior from different working directories - The command downloads to ./dasel (current directory) before moving. Ensure this works correctly regardless of where the script is executed from.

Test Plan

Run the script on a fresh Linux environment and verify:

  1. dasel installs successfully when not present
  2. The script skips installation when dasel is already installed
  3. The installed dasel binary is functional (dasel --version)

Notes

Co-Authored-By: yujonglee <yujonglee.dev@gmail.com>
@devin-ai-integration
Copy link
Contributor Author

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR. Add '(aside)' to your comment to have me ignore it.
  • Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

  • Disable automatic comment and CI monitoring

@netlify
Copy link

netlify bot commented Nov 23, 2025

Deploy Preview for hyprnote ready!

Name Link
🔨 Latest commit 0d2792a
🔍 Latest deploy log https://app.netlify.com/projects/hyprnote/deploys/6922b88476493900081a621b
😎 Deploy Preview https://deploy-preview-1810--hyprnote.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@coderabbitai
Copy link

coderabbitai bot commented Nov 23, 2025

Important

Review skipped

Bot user detected.

To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch devin/1763883073-add-dasel-installation

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

@yujonglee yujonglee merged commit af3ba9c into main Nov 23, 2025
6 checks passed
@yujonglee yujonglee deleted the devin/1763883073-add-dasel-installation branch November 23, 2025 07:40
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