refactor(composer): refine Airflow timeline styles and path hierarchy

[kyasbal]

Summary

Refactored the Airflow/Cloud Composer timeline hierarchy and updated timeline styling properties (icons, colors, priorities, and sort policies) to improve visual clarity on the UI. Also removed redundant GCP Project ID parameters from timeline path builders.

Key Changes

1. Timeline Styling and Registration

• Updated timeline type registrations in pkg/task/inspection/googlecloudclustercomposer/contract/timeline_type.go:
  • Updated Material icons, colors, display priorities, and sort policies (AlphabeticalSortPolicy, ChronologicalSortPolicy) for Airflow timelines (TimelineTypeAirflowDAG, TimelineTypeAirflowDAGRun, TimelineTypeAirflowTaskInstance, etc.).
  • Updated display category names and timeline styles to make the timeline tree easier to read.

2. Timeline Path Hierarchy Simplification

• Refactored timeline path helpers in pkg/task/inspection/googlecloudclustercomposer/contract/timeline_path.go:
  • Replaced MustComposerEnvironmentTimeline with MustAirflowTimeline, removing the unnecessary GCP Project ID parameter.
  • Standardized component timeline path creation via MustAirflowComponentTimeline.
  • Renamed the DAG Processor Manager timeline root to DAG files (MustAirflowDAGFilesTimeline).

3. Mapper Implementations and Test Updates

• Updated mappers and their corresponding unit tests to use the simplified TimelinePath helpers, removing unused ClusterIdentityTaskID dependencies:
  • pkg/task/inspection/googlecloudclustercomposer/impl/dag_processor_manager_mapper.go
  • pkg/task/inspection/googlecloudclustercomposer/impl/scheduler.go
  • pkg/task/inspection/googlecloudclustercomposer/impl/worker.go
  • pkg/task/inspection/googlecloudclustercomposer/impl/other.go

Verification

• Confirmed all package unit tests pass successfully:
  • go test ./pkg/task/inspection/googlecloudclustercomposer/...

[gemini-code-assist]

Code Review

This pull request refactors the timeline path and style definitions for Google Cloud Composer / Airflow. It replaces the project-level timeline hierarchy with a root-level Airflow timeline, simplifies component timeline creation, and updates various timeline types and styles. Additionally, it removes the unnecessary dependency on ClusterIdentityTaskID across multiple mappers. The review feedback highlights that several timeline styles use raw style.Color with manual float values, which is inconsistent with the codebase and can lead to incorrect color rendering on the UI. It is recommended to use style.MustForceConvertSRGBHex instead to ensure proper color space conversion.