[Gemini Log Analyzer] Part 4: Frontend Integration (Dashboard UI)

[ievdokdm]

This is Part 4 of the stacked Pull Request series implementing the Gemini Log Analyzer feature.

Depends on Part 3: #5043 (Note: Targets main due to GitHub cross-repo limits, but chains to #5043 via description).

This PR integrates the feature into the Cocoon Dashboard UI:

• Adds "Log Analysis" tab to display the Gemini log analyzer results when available.
• Adds "Analyze Logs with Gemini" action button to trigger the API from the presubmit dashboard.
• Implements spellchecking, permission-checking, and error-handling for the trigger action.

Stacked PR Series:

• Part 1: Backend Foundation (Models & Setup) (Merged)
• Part 2: Backend Infrastructure (Config & Database Data Flow) ([Gemini Log Analyzer] Part 2: Backend Infrastructure (Config, Models, & Database Flow) #5045)
• Part 3: Backend Core API (Genkit Logic & Handlers) ([Gemini Log Analyzer] Part 3: Backend Core (Analyze Logs API & Service) #5043)
• Part 4: Frontend Dashboard UI (This PR)

[jtmcdole]

No clue why there is a discrepancy between VSCode checkout of your branch and github's render:

presubmit_job.dart line 251:
q: Should you use int.tryParse just to avoid throwing if the field is unparsable?
you could also cheat and just do: int.tryParse(fields[fieldBuildId]?.integerValue ?? '')

app_dart/lib/src/model/firestore/presubmit_job.dart: line 314:

fields[fieldBuildId] = build.id.toValue();

app_dart/lib/src/request_handlers/analyze_logs.dart: line 59:
Can we validate this slug is in the configured repos (there is a set in Config)? I get UnifiedCheckRun.getLatestPresubmitGuardForPrNum might also check, but it would be nice to have this explicit.

app_dart/tool/local_server.dart: line 26:
you could allow someone working locally to test with their own key:

  secretManager.putString('APP_DART_GEMINI_LOG_ANALYZER_KEY', Platform.environment['APP_DART_GEMINI_LOG_ANALYZER_KEY'] ?? dummy_key);

packages/cocoon_integration_test/lib/src/fakes/fake_config.dart: line 183:
GenKit supports web (according to pub.dev) so you could use the following. Note, this is different from Platform and is instead a dart compile option (-DAPP_DART_GEMINI_LOG_ANALYZER_KEY="$APP_DART_GEMINI_LOG_ANALYZER_KEY")

      geminiLogAnalyzerKeyValue ?? 
      String.fromEnvironment('APP_DART_GEMINI_LOG_ANALYZER_KEY', defaultValue: 'fake-gemini-key');

[jtmcdole]

Note to self: I don't think we need @jsonkey on every field if we're not overriding the defaults.

[auto-submit]

auto label is removed for flutter/cocoon/5044, Failed to merge flutter/cocoon/5044 with Pull request flutter/cocoon/5044 could not be merged: Required status check "ci.yaml validation" is expected..