docs: fix inaccuracies vs. Enterprise Server ground truth (Azure DevOps extension, etc.)#258
Merged
Merged
Conversation
…round truth PR #256 imported 71 pages from a marketing source; several described components or config Pixee does not have. Validated all new docs against the Pixee Enterprise Server engineering docs (charts/pixee-enterprise-server-docs) and corrected the high-confidence factual errors: - "Azure DevOps extension" -> "Azure DevOps integration" (no such extension exists; ADO is a PAT + webhook integration). getting-started, faq. - GitLab token scopes completed (added ai_features, read_registry, read_virtual_registry) and Bitbucket scopes given the :bitbucket suffix. - PostgreSQL "15+" -> "17.4+". - Observability: removed fabricated Grafana / "Grafana k8s-monitoring" / "Grafana Cloud Private Data Connector"; the real stack is VictoriaMetrics/VictoriaLogs/VictoriaTraces with VMUI dashboards. - BYOM: removed unsupported providers (Google Vertex AI, Gemini, Nova, Mistral); reframed AWS Bedrock as the OpenAI-compatible path it is. - "Hierarchical LLM routing (7 named tiers)" -> actual reasoning/fast/SCA/ deep-research model slots. - Air-gap: "Zero outbound internet after installation" -> outbound limited to (proxyable) license validation, matching the rest of the docs. - SonarQube: removed invented "Browse and Administer" permissions; documented the required token scope + webhook secret. - Universal SARIF / CI-CD: fixed the fabricated upload endpoint (api.pixee.ai/v1/sarif/upload) to the real multipart POST /api/v1/repositories/$REPO_ID/scans, and removed the nonexistent "Azure DevOps Code Scanning" surface. yarn build passes clean (zero broken links). Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
ryandens
commented
Jun 24, 2026
Restore original wording (Grafana retained) in byom, enterprise-overview, observability, deployment-overview, security, and sonarqube per reviewer request. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
… table Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
PR #256 imported 71 pages from a marketing source (
Pixee-Marketing-OSPR #117), introducing references to components Pixee does not have — the flagged example being an "Azure DevOps extension."This PR was validated against
charts/pixee-enterprise-server-docs(the Pixee Enterprise Server engineering docs). After review, the scope was narrowed to the high-confidence, agreed-upon factual corrections only; speculative rewrites were reverted (Grafana, BYOM provider list, observability, SonarQube auth, air-gap wording, etc. are unchanged frommain).yarn buildpasses clean (zero broken links) andyarn check-format(prettier) passes.Changes in this PR
getting-started/getting-started.md,faq/faq.md(×2)getting-started/source-control.md,faq/faq.mdai_features,read_registry,read_virtual_registry) — 5 → 8, matching the GitLab Integration requirements.getting-started/source-control.md:bitbucketsuffix to all 6 scopes, matching the BitBucket Cloud Integration requirements.enterprise/deployment/helm.mdinstallation/prerequisites.md).integrations/sarif-universal.md; de-referenced inci-cd.md,enterprise/troubleshooting.md,integrations/integrations-overview.md,api/sarif.mdReverted after review (no longer in this PR)
Per reviewer direction, these were restored to their original
maincontent:byom.md(provider list),enterprise-overview.md("7 named tiers"),observability.md+deployment-overview.md(Grafana stack retained),platform/security.md(air-gap wording),integrations/scanners/sonarqube.md, and theci-cd.mdcontent edits (only the Universal SARIF reference was removed).Still worth a look (not changed here)
A few other ground-truth discrepancies were found but left for a separate product decision: the
PIXEE.yamlvsPIXEE.mdconfig-file question, the scoped-API-token model inapi/api-overview.md, the outbound-webhooks page (api/webhooks.md), a possible missing Black Duck page, and the Azure DevOps Server / Bitbucket Data Center support claims.Test plan
yarn buildclean — zero broken links (Universal SARIF page deletion verified)yarn check-format(prettier) passes🤖 Generated with Claude Code