Skip to content

Rewrite maifetch in Odin#3

Open
modelsbridgeaicom-ship-it wants to merge 2 commits into
HutchyBen:mainfrom
modelsbridgeaicom-ship-it:rewrite/zig-cli
Open

Rewrite maifetch in Odin#3
modelsbridgeaicom-ship-it wants to merge 2 commits into
HutchyBen:mainfrom
modelsbridgeaicom-ship-it:rewrite/zig-cli

Conversation

@modelsbridgeaicom-ship-it

@modelsbridgeaicom-ship-it modelsbridgeaicom-ship-it commented Jun 8, 2026

Copy link
Copy Markdown

/claim #1

Updated this PR from Zig to Odin after the latest maintainer direction.

Changes:

  • Replaces the previous Go/Zig implementation with an Odin CLI.
  • Preserves the existing access token, config file, logo size, and score count options.
  • Supports documented -a and legacy -t token flags.
  • Keeps MaiTea profiles/plays JSON parsing and the recent-score terminal output.
  • Uses curl for authenticated API GET requests so the Odin code stays small and portable.
  • Adds Odin tests for argument parsing, fullwidth text normalization, and score output formatting.

Verification:

  • odin check src -vet passes locally.
  • Full local link/test could not run in this Windows environment because the Odin compiler requires a Windows SDK that is not installed here.

@HutchyBen

HutchyBen commented Jun 8, 2026

Copy link
Copy Markdown
Owner

Actually upon further evaluation i've decided Odin would be the better choice for this project, upping the bounty to £100

@modelsbridgeaicom-ship-it modelsbridgeaicom-ship-it changed the title Rewrite maifetch in Zig Rewrite maifetch in Odin Jun 8, 2026
@modelsbridgeaicom-ship-it

Copy link
Copy Markdown
Author

Updated this PR to Odin per your latest direction. The branch now replaces the earlier Zig rewrite with an Odin CLI, keeps the config/env/CLI behavior, preserves recent-score output, and includes Odin tests around args, fullwidth normalization, and output formatting.

Local verification: odin check src -vet passes. I could not run the linker-backed odin test src / odin build src on this Windows box because the Windows SDK is missing, but the Odin frontend check is clean.

@HutchyBen

Copy link
Copy Markdown
Owner

Upon further investigation i have decided the language i want this project to be written in is Java 8, i am increasing the bounty to £150 for the inconvenience

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