Skip to content

fix(catalog): remove unavailable Nobulex source listing - #357

Merged
rajnisht7 merged 3 commits into
agentrust-io:mainfrom
dinakarjs:fix/unavailable-nobulex-listing
Oct 28, 2026
Merged

rajnisht7 merged 3 commits into
agentrust-io:mainfrom
dinakarjs:fix/unavailable-nobulex-listing

Conversation

@dinakarjs

@dinakarjs dinakarjs commented Oct 21, 2026 •

Copy link
Copy Markdown
Contributor

What and why

Remove the Nobulex integration listing whose source returned HTTP 451 / repository access blocked. CONTRIBUTING requires source links an anonymous reader can open. No link-check exemption or replacement product claim is introduced.

Current conflict repair

Synchronized with main 404001f in a signed merge commit, preserving the existing contribution history. Regenerated the integration index and marketplace catalog from the merged manifests, including the new PIC Standard MCP Guard entry. The net delta against main remains the same four listing/index files.

Validation

Published head e83f93e; tested tree e9acd7bafce710cd333b48cc33f0ba604bcc6232. Local Linux/Python 3.12: 49 manifest validations, compatibility validation, generated index/catalog checks and diff whitespace all pass. No runtime changes. No new external link crawl was performed; the earlier upstream restriction is historical evidence, not a fresh availability result. Hosted checks and independent current-head review remain separate.

Prepared with OpenAI Codex; DCO sign-offs retained.

Signed-off-by: Srinivasa Dinakar Jainti <dinakarjs@gmail.com>
@github-actions

github-actions Bot commented Oct 21, 2026 •

Copy link
Copy Markdown
Contributor

🟡 Contributor Check: MEDIUM

Check Result
Profile MEDIUM
Credential LOW
Overall MEDIUM

Automated check by AgenTrust Contributor Check.

Regenerate the index/catalog from the merged manifests; retain the complete current-main companion integration.

Signed-off-by: Srinivasa Consultants <78673153+dinakarjs@users.noreply.github.com>

Copy link
Copy Markdown
Contributor Author

Conflict resolved at d80d021 by merging current main and regenerating both index/catalog from the merged manifests. Local validation reports 48 integrations / zero failures; both generated-file checks and whitespace pass. Tested and published tree match 8fba46691333d95df0534bf3ad36dd1d8900bf3f; GitHub now reports mergeable.

The current-head hosted link check and integration validation pass. Examples still fails in WCM with the inherited model-signing import of missing OpenSSL; I verified that in failed job 119867882928. It is separately addressed by maintainer #362 / retained broader #360. This listing change does not weaken the example check, and I am not claiming the whole CI is green. Prepared with OpenAI Codex.

carloshvp
carloshvp previously approved these changes Oct 23, 2026

@carloshvp carloshvp left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Reviewed exact head d80d021f0594f7721e2b8674ec56dca410dd9b7d in an isolated checkout.

Approving removal of the unavailable listing. I independently queried the listed source today and GitHub returned HTTP 451 / Repository access blocked. The removal follows the contribution rule requiring publicly accessible source links and introduces no link-check exemption or replacement product claim.

Independent exact-head validation: integration schema validation reports 48 integrations and zero failures; compatibility validation, generated index --check and generated marketplace catalog --check all pass. The complete delta is confined to the listing, its README/index row and generated catalog; the merged cA2A companion entry remains present. Whitespace checks pass.

The current-head hosted link check succeeds. The separate WCM job fails on its inherited missing OpenSSL import, confirmed from the log; #362 supplies that workflow repair. Normal CI and review gates remain applicable. Both PR commits carry DCO sign-off.

Reviewed with OpenAI Codex.

@rajnisht7

Copy link
Copy Markdown
Member

@dinakarjs Kindly resolve conflicts and rebase on main

Regenerate index and catalog while preserving merged PIC integration.
Prepared with OpenAI Codex.

Signed-off-by: Srinivasa Consultants <78673153+dinakarjs@users.noreply.github.com>

Copy link
Copy Markdown
Contributor Author

@rajnisht7 conflict resolved at e83f93e by merging current main 404001f and regenerating the index/catalog. This preserves the existing commits and the newly merged PIC integration. All 49 manifest validations, compatibility, both generated-file checks and whitespace pass locally; submitted tree matches the tested tree. Please re-review this head. Hosted validation remains separate. Prepared with OpenAI Codex.

@rajnisht7
rajnisht7 requested a review from carloshvp October 27, 2026 19:06
@noah-ing

Copy link
Copy Markdown
Collaborator

The refreshed current-head link jobs for #359 and #360 still fail on the same two Nobulex URLs (HTTP 451) that this PR removes; #374 has the same blocker. This PR's current head e83f93ec1300e9a6cca5214079ff7759f4fa9397 is mergeable, its link job passes, and Rajnish has approved it. Could a maintainer land #357 when ready? The dependent PRs can then be refreshed and checked without bypassing the links gate.

@rajnisht7
rajnisht7 merged commit 2a10687 into agentrust-io:main Oct 28, 2026
35 of 37 checks passed
noah-ing added a commit to noah-ing/integrations that referenced this pull request Oct 28, 2026
Signed-off-by: Noah Ingwers <98993329+noah-ing@users.noreply.github.com>
dinakarjs added a commit to dinakarjs/integrations that referenced this pull request Oct 29, 2026
Preserve the successor history and unchanged harness/probe locks while incorporating merged agentrust-io#357 and current validation baseline.

Prepared with OpenAI Codex.

Signed-off-by: Srinivasa Consultants <78673153+dinakarjs@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

needs-review:MEDIUM Contributor check flagged MEDIUM risk

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants