Skip to content

registry: add flux-operator & flux-operator-mcp#8852

Merged
jdx merged 5 commits intojdx:mainfrom
monotek:add-flux-operator
Apr 21, 2026
Merged

registry: add flux-operator & flux-operator-mcp#8852
jdx merged 5 commits intojdx:mainfrom
monotek:add-flux-operator

Conversation

@monotek
Copy link
Copy Markdown
Contributor

@monotek monotek commented Apr 2, 2026

  • add flux-operator
  • add flux-operator-mcp

@monotek monotek changed the title add flux-operator & flux-operator-mcp registry - add flux-operator & flux-operator-mcp Apr 2, 2026
@monotek monotek changed the title registry - add flux-operator & flux-operator-mcp registry: add flux-operator & flux-operator-mcp Apr 2, 2026
@greptile-apps
Copy link
Copy Markdown
Contributor

greptile-apps Bot commented Apr 2, 2026

Greptile Summary

This PR adds two new registry/*.toml entries for flux-operator and flux-operator-mcp, resolving a previously raised concern about missing registry files. The flux-operator.toml entry looks correct — it references a backend that has a corresponding aqua registry YAML (crates/aqua-registry/aqua-registry/pkgs/controlplaneio-fluxcd/flux-operator/registry.yaml) and follows the same test pattern as flux2.toml. The flux-operator-mcp.toml entry remains blocked pending a matching aqua registry YAML for controlplaneio-fluxcd/flux-operator-mcp, which does not yet exist in the local registry.

Confidence Score: 4/5

Safe to merge for flux-operator only; flux-operator-mcp will fail to install until its aqua registry YAML is present

flux-operator.toml is correct and ready. flux-operator-mcp.toml references a backend (aqua:controlplaneio-fluxcd/flux-operator-mcp) with no corresponding aqua registry YAML in the local registry — this is a present defect that will cause install failures for that tool.

registry/flux-operator-mcp.toml — depends on a missing aqua registry package definition

Important Files Changed

Filename Overview
registry/flux-operator.toml New registry entry for flux-operator; correctly references the existing aqua registry package and follows the same test pattern as flux2.toml
registry/flux-operator-mcp.toml New registry entry for flux-operator-mcp; references aqua:controlplaneio-fluxcd/flux-operator-mcp but no matching aqua registry yaml exists at crates/aqua-registry/aqua-registry/pkgs/controlplaneio-fluxcd/flux-operator-mcp/ — backend resolution will fail at install time

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A[mise install flux-operator] --> B[registry/flux-operator.toml]
    B --> C[backend: aqua:controlplaneio-fluxcd/flux-operator]
    C --> D[crates/aqua-registry/.../flux-operator/registry.yaml ✅]
    D --> E[Downloads GitHub release asset]

    F[mise install flux-operator-mcp] --> G[registry/flux-operator-mcp.toml]
    G --> H[backend: aqua:controlplaneio-fluxcd/flux-operator-mcp]
    H --> I[crates/aqua-registry/.../flux-operator-mcp/registry.yaml ❌ MISSING]
    I --> J[Resolution fails]
Loading

Reviews (10): Last reviewed commit: "Merge branch 'main' into add-flux-operat..." | Re-trigger Greptile

Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request adds registry definitions for flux-operator and flux-operator-mcp. The feedback suggests explicitly defining the name field in both files to ensure correct binary identification and consistency.

@monotek monotek force-pushed the add-flux-operator branch from 50b1d5d to efd6666 Compare April 2, 2026 16:31
@monotek monotek marked this pull request as draft April 2, 2026 16:57
Signed-off-by: André Bauer <monotek23@gmail.com>
@monotek monotek force-pushed the add-flux-operator branch from 89e2f37 to 803acdb Compare April 2, 2026 17:04
@monotek monotek marked this pull request as ready for review April 2, 2026 17:07
@jdx
Copy link
Copy Markdown
Owner

jdx commented Apr 4, 2026

The aqua registry YAML files under crates/aqua-registry/ are vendored from the upstream aqua-registry and should not be modified directly in this repo. Please submit the package definitions to the upstream aqua-registry instead, and they will be picked up here when we next update the vendored copy.

The registry/*.toml files are fine to add here, but the backend should reference the package after it's been accepted upstream.

This comment was generated by Claude Code.

@jdx jdx marked this pull request as draft April 4, 2026 15:41
@monotek
Copy link
Copy Markdown
Contributor Author

monotek commented Apr 7, 2026

Thanks for the info.
I've deleted the files.

Create a PR in aqua registry: aquaproj/aqua-registry#51667
I failed to use "argd" for the mcp part though.

I'll give it a try again if i have more time...

@monotek
Copy link
Copy Markdown
Contributor Author

monotek commented Apr 17, 2026

flux-operator was merged.
added flux-operator-mcp as well: aquaproj/aqua-registry#52237

Comment thread registry/flux-operator-mcp.toml
@monotek
Copy link
Copy Markdown
Contributor Author

monotek commented Apr 21, 2026

flux-operator-mcp is merged too in aqua regsitry: aquaproj/aqua-registry#52237

@monotek monotek marked this pull request as ready for review April 21, 2026 13:12
@jdx jdx merged commit 1db3479 into jdx:main Apr 21, 2026
35 checks passed
jdx pushed a commit that referenced this pull request Apr 22, 2026
### 🚀 Features

- **(backend)** support aqua vars templates by @risu729 in
[#9110](#9110)
- add gsudo (Sudo for Windows) to registry by @matracey in
[#9281](#9281)

### 🐛 Bug Fixes

- **(cli)** retrieve token from github helper for `self-update` command
by @sushichan044 in [#9259](#9259)
- **(github)** scope auth headers to API URLs by @risu729 in
[#9271](#9271)
- **(vfox)** use github token for lua http requests by @jdx in
[#9257](#9257)

### 📚 Documentation

- add aube hero banner by @jdx in
[#9265](#9265)
- add en.dev footer by @jdx in
[#9267](#9267)
- implement landing page design by @jdx in
[#9266](#9266)

### 📦️ Dependency Updates

- lock file maintenance by @renovate[bot] in
[#9268](#9268)

### 📦 Registry

- add llama.cpp
([github:ggml-org/llama.cpp](https://github.com/ggml-org/llama.cpp)) by
@igor-makarov in [#9282](#9282)
- add kiro-cli by @shalk in
[#9274](#9274)
- add flux-operator & flux-operator-mcp by @monotek in
[#8852](#8852)

### Chore

- **(release)** add en.dev sponsor blurb to release notes by @jdx in
[#9272](#9272)
- bump communique to 1.0.1 by @jdx in
[#9264](#9264)

### New Contributors

- @monotek made their first contribution in
[#8852](#8852)
- @igor-makarov made their first contribution in
[#9282](#9282)

## 📦 Aqua Registry Updates

#### New Packages (2)

-
[`controlplaneio-fluxcd/flux-operator/flux-operator-mcp`](https://github.com/controlplaneio-fluxcd/flux-operator/flux-operator-mcp)
- [`endevco/aube`](https://github.com/endevco/aube)

#### Updated Packages (1)

- [`graelo/pumas`](https://github.com/graelo/pumas)
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