fix(cli) Fixing helix push

[xav-db]

Greptile Overview

Greptile Summary

Fixed container name display format and improved container existence checking in the CLI, and resolved HQL codegen failure for WHERE clauses with traversal steps before property access.

Key Changes

• CLI Container Naming: Updated display format from helix_{project}_{instance} to helix-{project}-{instance} to match actual Docker container naming convention (hyphens instead of underscores)
• Container Check Improvement: Refactored instance_exists to use centralized container_name method and exact string matching instead of contains, improving accuracy
• HQL Codegen Fix (bug (hql): WHERE(_::ToN::ID::EQ(...)) Causes Rust Codegen Failure #847): Added support for WHERE clauses with multi-step traversal patterns like _::ToN::ID::EQ(id) and _::ToN::{age}::EQ(age) that previously caused Rust codegen failures
• Test Coverage: Added comprehensive test suite for the new traversal property access patterns with 6 test queries covering various edge cases

The changes are well-structured and include proper test coverage for the codegen fix.

Important Files Changed

Filename	Overview
helix-cli/src/commands/push.rs	Fixed container name display format from underscores to hyphens to match actual naming scheme
helix-cli/src/docker.rs	Improved container existence check to use centralized container_name method and exact match instead of contains
helix-db/src/helixc/generator/traversal_steps.rs	Added codegen support for WHERE clauses with traversal steps before property access (fixes #847)

Sequence Diagram

sequenceDiagram
    participant User
    participant CLI as Helix CLI
    participant Docker as Docker/Podman
    participant Compiler as HelixC Codegen
    
    Note over User,Compiler: CLI Container Naming Fix
    User->>CLI: helix push
    CLI->>CLI: compose_project_name(instance_name)
    Note right of CLI: Returns "helix-{project}-{instance}"
    CLI->>CLI: container_name(instance_name)
    Note right of CLI: Returns "helix-{project}-{instance}_app"
    CLI->>Docker: Check container exists
    Docker-->>CLI: Container status
    CLI->>CLI: Display container info
    Note right of CLI: Now shows "helix-{project}-{instance}" (fixed from "helix_{project}_{instance}")
    
    Note over User,Compiler: HQL Traversal Property Access Fix (#847)
    User->>Compiler: Compile WHERE(_::ToN::ID::EQ(id))
    Compiler->>Compiler: Parse traversal steps
    Note right of Compiler: Detects pattern: [ToN] + [ID] + [EQ]
    Compiler->>Compiler: Generate optimized filter_ref
    Note right of Compiler: Creates traversal chain + property access
    Compiler-->>User: Valid Rust code generated

Loading