Skip to content

Add various tools suggested by ChatGPT#1245

Open
alexellis wants to merge 1 commit intomasterfrom
add_various
Open

Add various tools suggested by ChatGPT#1245
alexellis wants to merge 1 commit intomasterfrom
add_various

Conversation

@alexellis
Copy link
Copy Markdown
Owner

Description

Add various tools suggested by ChatGPT

Motivation and Context

Adds various popular tools suggested via ChatGPT that have static binaries on GitHub

How Has This Been Tested?

amp ran the usual tests with --os / --arch etc.

  • tldr (tealdeer-rs/tealdeer): A very fast implementation of tldr in Rust.
  • micro (micro-editor/micro): A modern and intuitive terminal-based text editor.
  • zellij (zellij-org/zellij): A terminal workspace with batteries included.
  • yazi (sxyazi/yazi): Blazing fast terminal file manager written in Rust.
  • lf (gokcehan/lf): Terminal file manager with vim-like bindings.
  • jless (PaulJuliusMartinez/jless): A command-line JSON viewer designed for reading, exploring, and searching through JSON data.
  • trufflehog (trufflesecurity/trufflehog): Find and verify credentials in code repositories.
  • gitleaks (gitleaks/gitleaks): Protect and discover secrets using Gitleaks.
  • sesh (joshmedeski/sesh): Smart session manager for the terminal.
  • jj (jj-vcs/jj): A Git-compatible VCS that is both simple and powerful.
  • mcfly (cantino/mcfly): Fly through your shell history with context-aware suggestions.
  • pueue (Nukesor/pueue): Manage your shell commands as a task queue.
  • doggo (mr-karan/doggo): Command-line DNS client for humans. Features include DoH, DoT, DoQ support.
  • oha (hatoo/oha): HTTP load generator, inspired by rakyll/hey with tui animation.
  • fq (wader/fq): jq for binary formats - tool, language and decoders for working with binary data.
  • sd (chmln/sd): Intuitive find & replace CLI (sed alternative).

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Documentation

  • I have updated the list of tools in README.md if (required) with ./arkade get --format markdown
  • I have updated the list of apps in README.md if (required) with ./arkade install --help

Signed-off-by: Alex Ellis (OpenFaaS Ltd) <alexellis2@gmail.com>
@reviewfn
Copy link
Copy Markdown

reviewfn bot commented Jan 22, 2026

AI Pull Request Overview

Summary

  • Adds 16 new CLI tools to arkade, expanding the toolset from 187 to 203 tools
  • Tools include terminal utilities, file managers, text editors, security scanners, and development aids
  • Updates README.md with alphabetical tool listings and updated count
  • Implements comprehensive test coverage for URL generation across platforms
  • Adds proper binary templates for cross-platform downloads

Approval rating (1-10)

5/10 - Technically sound implementation but lacks critical evaluation of tool selection criteria, security implications, and long-term maintenance.

Summary per file

Summary per file
File path Summary
README.md Updated tool list with 16 new entries and count from 187 to 203
pkg/get/get_test.go Added 8 new test functions covering URL generation for all new tools
pkg/get/tools.go Added 16 new Tool structs with binary templates for cross-platform support

Overall Assessment

The PR successfully expands arkade's tool ecosystem by adding popular CLI utilities, demonstrating good technical implementation with proper testing and platform support. However, the approach of adding tools based on ChatGPT suggestions without documented evaluation criteria raises significant concerns about tool quality, security, licensing, and maintenance burden. Security tools like trufflehog and gitleaks require particular scrutiny for trustworthiness and official distribution channels.

Detailed Review

Detailed Review

README.md

  • Tool list maintains alphabetical order correctly
  • Count updated accurately from 187 to 203
  • Descriptions match those in tool definitions
  • No issues with formatting or links

pkg/get/get_test.go

  • Test functions follow existing patterns with proper naming (Test_Download{ToolName})
  • Coverage includes linux/darwin/Windows and x86_64/arm64 architectures where applicable
  • URL templates appear correct based on GitHub release patterns
  • No issues with test structure or assertions

pkg/get/tools.go

  • All new tools added at the end of the MakeTools function
  • Binary templates handle OS/architecture variations appropriately
  • Issue: zellij binary template lacks Windows support - only handles darwin and linux, missing "ming" case
  • Issue: jless binary template lacks Windows support - only handles darwin and linux
  • Issue: mcfly binary template lacks arm64 support on darwin - only handles x86_64 for macOS
  • Issue: pueue binary template lacks arm64 support on darwin - only handles x86_64 for macOS
  • Risk: Security tools (trufflehog, gitleaks) added without verification of official distribution channels or security audits
  • Risk: No evaluation of tool licenses, maintenance status, or community adoption
  • Risk: Potential for supply chain attacks through unofficial binaries
  • Suggestion: Add Windows support for zellij and jless if available, or document platform limitations
  • Suggestion: Implement arm64 support for mcfly and pueue on macOS if binaries exist
  • Recommendation: Establish criteria for tool inclusion including security review, license compatibility, and maintenance status
  • Recommendation: Consider phased rollout or community feedback for new tool additions

AI agent details.

Agent processing time: 45.402s
Environment preparation time: 6.944s
Total time from webhook: 56.165s

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.

1 participant