Skip to content

feat: Enhance integration setup guides and improve UI components - #3010

Merged
AlexSanchez-bit merged 2 commits into
release/v12.0.0from
backlog/v12-integration-guide
Oct 29, 2026
Merged

AlexSanchez-bit merged 2 commits into
release/v12.0.0from
backlog/v12-integration-guide

Conversation

@AndresTK89

Copy link
Copy Markdown
  • Added support for displaying guide settings in the IntegrationDrawer component.
  • Refactored ChannelGuide to utilize ForwarderGuide for better configuration management.
  • Updated CustomSetup to handle missing data types and improve user feedback.
  • Enhanced ForwarderGuide to support default ports and protocol selection.
  • Introduced GenericCollectorGuide for better handling of collector integrations.
  • Improved RemoteEnablePanel to utilize default ports more effectively.
  • Updated injectedModules to include forwarder data types and improved metadata handling.
  • Enhanced integration tests to ensure proper guide metadata assignment and deduplication.
  • Added new localization strings for setup guides in English and Spanish.

- Added support for displaying guide settings in the IntegrationDrawer component.
- Refactored ChannelGuide to utilize ForwarderGuide for better configuration management.
- Updated CustomSetup to handle missing data types and improve user feedback.
- Enhanced ForwarderGuide to support default ports and protocol selection.
- Introduced GenericCollectorGuide for better handling of collector integrations.
- Improved RemoteEnablePanel to utilize default ports more effectively.
- Updated injectedModules to include forwarder data types and improved metadata handling.
- Enhanced integration tests to ensure proper guide metadata assignment and deduplication.
- Added new localization strings for setup guides in English and Spanish.
Signed-off-by: Andres Aguilera <andres@insecureweb.com>
@github-actions

Copy link
Copy Markdown

✅ AI review — Minor findings

Only minor (medium/low) issues were found. Consider addressing them.

⚠️ architecture (silas-1.7-pro) — minor findings

Summary: Frontend integration guide refactor duplicates port/protocol resolution and overwrites backend guideOnly semantics; no agent or wire-contract critical path touched.

  • medium frontend/src/features/integrations/constants/injectedModules.ts:180 — applyInjectedGuideMetadata unconditionally sets guideOnly: true for backend rows, conflating real catalog integrations with frontend demo fallbacks. Preserve integration.guideOnly or set it only for injected rows without a backend id.
  • medium frontend/src/features/integrations/components/setup/collector/GenericCollectorGuide.tsx:14 — getDefaultEndpoint duplicates protocol/port resolution logic already present in protoCatalog, ForwarderGuide, and RemoteEnablePanel. Extract a shared endpoint-resolution helper to avoid divergent default behavior.
  • medium frontend/src/features/integrations/constants/injectedModules.ts:145 — INJECTED_MODULES throws at module load when guide metadata is missing, turning data validation into a runtime crash. Validate in tests or fail gracefully with a warning instead of coupling app startup to seed metadata completeness.
  • low frontend/src/features/integrations/pages/IntegrationsPage.tsx:47 — Kind is now derived from backend module.ingestType while category/KIND_META remains another source of truth. Prefer one canonical mapping for integration kind to avoid inconsistent UI grouping.
  • low frontend/src/features/integrations/components/setup/ChannelGuide.tsx:45 — SyslogDeviceGuide and NetflowExporterGuide duplicate collector selection state and host fallback logic. Consider a shared wrapper around ForwarderGuide for channel-specific guides.
  • low frontend/src/features/integrations/components/setup/collector/ForwarderGuide.tsx:180 — fallbackPort combines defaultPortFor, defaultPort, and port, while RemoteEnablePanel also applies defaultPort fallback. Centralize initial port selection to keep ForwarderGuide and RemoteEnablePanel consistent.

✅ bugs (silas-1.7-pro) — clean

Summary: No concrete bugs found in the visible diff. Changes refactor collector guides to use ForwarderGuide with dynamic selection, restructure injected module metadata, and add i18n strings. All logic paths are guarded and tested.

No findings.

✅ security (silas-1.7-pro) — clean

Summary: No vulnerabilities or customer-facing information disclosures introduced by the diff.

No findings.

🔴 go-deps — pending updates

🔍 Discovered 31 Go projects

📦 Dependencies with updates available:

  📁 ./agent:
     - github.com/threatwinds/go-sdk: v1.1.28 → v1.1.38
     - google.golang.org/grpc: v1.84.0 → v1.85.0

  📁 ./tools/rulecheck:
     - github.com/threatwinds/go-sdk: v1.1.31 → v1.1.38

  📁 ./agent-manager:
     - github.com/threatwinds/go-sdk: v1.1.31 → v1.1.38
     - google.golang.org/grpc: v1.84.0 → v1.85.0

  📁 ./collectors/forwarder:
     - github.com/threatwinds/go-sdk: v1.1.31 → v1.1.38
     - google.golang.org/grpc: v1.84.0 → v1.85.0

  📁 ./collectors/as400:
     - github.com/threatwinds/go-sdk: v1.1.31 → v1.1.38
     - google.golang.org/grpc: v1.84.0 → v1.85.0

  📁 ./collectors/utmstack:
     - github.com/threatwinds/go-sdk: v1.1.31 → v1.1.38
     - google.golang.org/grpc: v1.84.0 → v1.85.0

  📁 ./backend:
     - github.com/aws/aws-sdk-go-v2/config: v1.33.11 → v1.33.12
     - github.com/aws/aws-sdk-go-v2/credentials: v1.20.11 → v1.20.12
     - github.com/aws/aws-sdk-go-v2/service/sts: v1.51.5 → v1.51.6
     - github.com/crowdstrike/gofalcon: v0.23.0 → v0.24.0
     - google.golang.org/api: v0.309.0 → v0.310.0
     - google.golang.org/grpc: v1.84.0 → v1.85.0

  📁 ./plugins/aws:
     - github.com/aws/aws-sdk-go-v2: v1.47.4 → v1.47.5
     - github.com/aws/aws-sdk-go-v2/config: v1.33.11 → v1.33.12
     - github.com/aws/aws-sdk-go-v2/credentials: v1.20.11 → v1.20.12
     - github.com/aws/aws-sdk-go-v2/service/cloudwatchlogs: v1.90.2 → v1.90.3

  📁 ./plugins/gcp:
     - google.golang.org/api: v0.309.0 → v0.310.0

  📁 ./plugins/crowdstrike:
     - github.com/crowdstrike/gofalcon: v0.23.0 → v0.24.0

  📁 ./log-input:
     - google.golang.org/grpc: v1.84.0 → v1.85.0

❌ Please update dependencies before merging.

@AlexSanchez-bit
AlexSanchez-bit merged commit b65aedc into release/v12.0.0 Oct 29, 2026
1 check passed
@AlexSanchez-bit
AlexSanchez-bit deleted the backlog/v12-integration-guide branch October 29, 2026 17:22
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