Skip to content

docs(trustguard): GitHub Copilot is a real collector type - #219

Merged
NTSergiVidal merged 1 commit into
developfrom
sergividal/run-1283-copilot-collector-exists
Sep 8, 2026
Merged

docs(trustguard): GitHub Copilot is a real collector type#219
NTSergiVidal merged 1 commit into
developfrom
sergividal/run-1283-copilot-collector-exists

Conversation

@NTSergiVidal

Copy link
Copy Markdown
Contributor

Correction to #218. It claimed there was no GitHub Copilot entry in the catalog and told readers to ask NeuralTrust to enable one. That is wrong — the entry ships in the console today, as the catalog screenshots show.

What I got wrong

I audited the app repo against origin/develop, but the deployed console runs main. fix(collectors): add GitHub Copilot to the collectors catalog (#3496) landed on main and was never back-merged to develop, and it is the only difference between the two branches in collectorProviders.ts.

I even checked the app checkout was current before trusting it — but "current" for the product means main, not develop. Same class of re-baseline error I had just flagged for the docs repo, made in the other direction.

The real catalog

Catalog group Types
Gateway 6
AI assistants & coding agents 5
Application 5
WAF 4
Total 20

Changes

  • github-copilot.mdx — restores the real path, Catalog → AI assistants & coding agents → GitHub Copilot, replacing the false "no entry yet, ask NeuralTrust" claim.
  • trustguard/overview.mdx — Copilot added to its group, nineteen → twenty, and removed from the "not collector types" paragraph.
  • collectors.mdx — twenty types, the group's count 4 → 5, and Copilot dropped from the not-a-collector note.
  • Inference Hooks now described as the console card describes it — Claude chat and Cowork. Claude Code has its own catalog entry, so listing it under Inference Hooks in the overview was misleading. The source.application values on the Claude Enterprise page are unchanged and still correct.

Verification

Now run against the catalog extracted from app main, not develop:

catalogo (app main): 20
docu (overview):     20
>>> COINCIDENCIA 100%

0 broken catalog breadcrumbs, check-navigation.py passes with 169 entries.

Worth fixing outside the docs

#3496 is on app main but not on app develop, so the next develop→main release could drop GitHub Copilot from the catalog again. Someone on the app side should back-merge it.

Correction. The previous PR claimed there was no GitHub Copilot entry in the
catalog and told readers to ask NeuralTrust to enable one. That is wrong: the
entry exists and ships in the console today.

The mistake was auditing the app repo against origin/develop while the deployed
console runs main, and `fix(collectors): add GitHub Copilot to the collectors
catalog (#3496)` landed on main without a back-merge to develop. It is the only
difference between the two in that file.

The catalog therefore holds twenty collector types, not nineteen, with five in
the AI assistants & coding agents group. Restores the real console path on the
Copilot page, adds it to the overview table and the group count, and drops it
from the list of integrations that are not collector types.

Also aligns the Inference Hooks description with the console card, which says
Claude chat and Cowork — Claude Code has its own catalog entry.
@github-actions

github-actions Bot commented Sep 8, 2026

Copy link
Copy Markdown

Trivy code scan — HIGH/CRITICAL

No HIGH or CRITICAL vulnerabilities found. (run)

@NTSergiVidal
NTSergiVidal merged commit 9a92b6d into develop Sep 8, 2026
9 checks passed
@NTSergiVidal
NTSergiVidal deleted the sergividal/run-1283-copilot-collector-exists branch September 8, 2026 12:23
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