feat: add gurgeous/tennis

[hituzi-no-sippo]

gurgeous/tennis: stylish CSV tables in your terminal

$ aqua g -i gurgeous/tennis

Check List

• Read CONTRIBUTING.md
  • Read OSS Contribution Guide
• All commits are signed
  • This repository enables Require signed commits, so all commits must be signed
• Avoid force push
• Do only one thing in one Pull Request
• Execute cmdx s to scaffold code
• Install and execute the package and confirm if the package works well

How to confirm if this package works well

Reviewers aren't necessarily familiar with this package, so please describe how to confirm if this package works well.
Please confirm if this package works well yourself as much as possible.

Command and output

$ cat foo.csv 
foo,bar,baz
val1,val2,val3
$ tennis foo.csv
╭──────┬──────┬──────╮
│ foo  │ bar  │ baz  │
├──────┼──────┼──────┤
│ val1 │ val2 │ val3 │
╰──────┴──────┴──────╯
$

If files such as configuration file are needed, please share them.

None

Reference

• tennis: GitHub Release

Summary by CodeRabbit

• New Features
  • Added support for the tennis package (v0.0.2) in the package registry, enabling installation on Linux and Darwin ARM64 environments.

[coderabbitai]

📝 Walkthrough

Walkthrough

New package configuration files are added for the "tennis" project from the gurgeous organization. The changes introduce a package manifest and registry definitions specifying how to source and resolve the tennis binary from GitHub releases across Linux and Darwin ARM64 environments.

Changes

Cohort / File(s)	Summary
Tennis Package Configuration pkgs/gurgeous/tennis/pkg.yaml, pkgs/gurgeous/tennis/registry.yaml	New package manifest and registry definition for gurgeous/tennis@v0.0.2 with GitHub release sourcing configuration, asset templates, checksum validation, and platform support for linux and darwin/arm64.
Root Registry Update registry.yaml	Added new github_release package entry for gurgeous/tennis alongside existing hostctl provider, with version constraints, asset resolution, and checksum handling.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Suggested labels

enhancement

Poem

🐰 A tennis ball bounces into the registry,
New configs in place, no mystery,
From gurgeous's court it shall release,
Linux and Darwin at peace,
The rabbit serves up this package with ease!

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title 'feat: add gurgeous/tennis' clearly and concisely summarizes the main change: adding a new package to the registry.
Description check	✅ Passed	The description includes the checklist with all items marked complete, installation instructions, verification steps with actual command output, and references, meeting template requirements.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment

📝 Coding Plan

• Generate coding plan for human review comments

---

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

• X
• Mastodon
• Reddit
• LinkedIn

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

Tip

CodeRabbit can approve the review once all CodeRabbit's comments are resolved.

Enable the reviews.request_changes_workflow setting to automatically approve the review once all CodeRabbit's comments are resolved.

[coderabbitai]

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents

Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@registry.yaml`:
- Around line 43513-43515: The supported_envs list currently uses a broad
"linux" entry which allows all Linux architectures; update the supported_envs
array to list the explicit Linux architectures the upstream provides (e.g.
replace "linux" with "linux/amd64" and "linux/arm64", keeping "darwin/arm64"
as-is) so installers on unsupported architectures (armv7, s390x, etc.) are not
erroneously considered supported.

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 4cf90ecd-c1cb-4fbf-9abd-8dbd27a1409e

📥 Commits

Reviewing files that changed from the base of the PR and between 6d3500f and 221cd20.

📒 Files selected for processing (3)

• pkgs/gurgeous/tennis/pkg.yaml
• pkgs/gurgeous/tennis/registry.yaml
• registry.yaml

[coderabbitai]

⚠️ Potential issue | 🟠 Major

🧩 Analysis chain

🏁 Script executed:

#!/bin/bash
set -euo pipefail

# Verify actual published artifact matrix from upstream release.
repo="gurgeous/tennis"
tag="v0.0.2"

json="$(curl -fsSL "https://api.github.com/repos/${repo}/releases/tags/${tag}")"

echo "Assets:"
echo "$json" | jq -r '.assets[].name'

echo
echo "Derived supported_envs from asset names:"
echo "$json" \
  | jq -r '.assets[].name' \
  | sed -nE 's/^tennis_[0-9.]+_([a-z]+)_([a-z0-9_]+)\.tar\.gz$/\1\/\2/p' \
  | sed 's/x86_64/amd64/g; s/aarch64/arm64/g' \
  | sort -u

Repository: aquaproj/aqua-registry

Length of output: 276

---

Narrow supported_envs for Linux to explicit architectures.

The linux entry accepts all Linux architectures, but the upstream release only provides binaries for Linux amd64 and arm64. This can cause install attempts on unsupported architectures like arm7, s390x, etc.

🔧 Proposed fix

         supported_envs:
-          - linux
+          - linux/amd64
+          - linux/arm64
           - darwin/arm64

📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change

	supported_envs:
	- linux
	- darwin/arm64
	supported_envs:
	- linux/amd64
	- linux/arm64
	- darwin/arm64

🤖 Prompt for AI Agents

Verify each finding against the current code and only fix it if needed.

In `@registry.yaml` around lines 43513 - 43515, The supported_envs list currently
uses a broad "linux" entry which allows all Linux architectures; update the
supported_envs array to list the explicit Linux architectures the upstream
provides (e.g. replace "linux" with "linux/amd64" and "linux/arm64", keeping
"darwin/arm64" as-is) so installers on unsupported architectures (armv7, s390x,
etc.) are not erroneously considered supported.

[suzuki-shunsuke]

Thank you!