Failed to generate code suggestions for PR

Greptile Overview

Greptile Summary

This PR hides the service graph feature from the UI by adding a configuration flag (service_graph_enabled) that controls visibility of the service-maps tab in the traces interface.

Changes:

• Backend exposes service_graph_enabled configuration field (true for enterprise with feature enabled, false otherwise)
• Frontend conditionally renders service-maps tab toggle buttons based on the config flag
• URL handling prevents navigation to service-maps tab when feature is disabled
• Service-maps content is hidden when the feature is disabled

Minor issue:

• The activeTab watcher mutates its own watched value, causing a double re-render (not critical, but not ideal for performance)

Confidence Score: 4/5

• This PR is safe to merge with minor optimization opportunity
• The implementation correctly hides the service graph feature through configuration flags. The backend changes are straightforward with proper feature flag usage. Frontend changes properly handle conditional rendering and URL state. One minor style issue exists where the activeTab watcher mutates its own watched value causing unnecessary re-renders, but this doesn't affect functionality.
• web/src/plugins/traces/Index.vue has a minor optimization opportunity in the activeTab watcher

Important Files Changed

File Analysis

Sequence Diagram

sequenceDiagram
    participant User
    participant Browser
    participant Frontend
    participant Backend
    participant Config
    
    User->>Browser: Navigate to Traces page
    Browser->>Frontend: Load Index.vue
    Frontend->>Backend: GET /api/default/config
    Backend->>Config: Check service_graph.enabled
    alt Enterprise build
        Config-->>Backend: o2cfg.service_graph.enabled
    else Non-enterprise build
        Config-->>Backend: false
    end
    Backend-->>Frontend: ConfigResponse{service_graph_enabled}
    Frontend->>Frontend: Store config in Vuex (store.state.zoConfig)
    
    alt service_graph_enabled = true
        Frontend->>Frontend: Render SearchBar with tab toggle buttons
        Frontend->>Frontend: Allow service-maps tab selection
    else service_graph_enabled = false
        Frontend->>Frontend: Hide tab toggle buttons (v-if)
        Frontend->>Frontend: Hide service-maps content (v-if)
        alt URL has ?tab=service-maps
            Frontend->>Frontend: onBeforeMount: Force activeTab = 'search'
        end
    end
    
    Frontend-->>User: Display traces page (with or without service-maps tab)