rover dev not respecting path configuration in router.yaml

[westhechiang]

Description

When using rover dev with --router-config pointed at a router.yaml where path is set to /graphql, the path configuration is ignored and the supergraph endpoint ends up being localhost:3000.

Steps to reproduce

# sample rover dev command
APOLLO_ELV2_LICENSE=accept rover dev --name my-app --schema my-app/schema.gql --url http://localhost:6002/graphql --router-config ./router.local.yaml

# sample router.yaml
supergraph:
  # The path for GraphQL execution
  # (Defaults to /)
  path: /graphql
  listen: 0.0.0.0:3000
  introspection: false
homepage:
  enabled: true
sandbox:
  enabled: false
cors:
  origins:
    - [REDACTED]
  allow_credentials: true
  methods:
    - GET
    - POST
    - OPTIONS
headers:
  all:
    request:
      - propagate:
          named: 'cookie'
      - propagate:
          named: 'host'
          rename: 'X-Forwarded-Host'
      - propagate:
          named: [REDACTED]
  subgraphs:
    my-app:
      request:
        - propagate:
            named: 'x-csrf-token'
telemetry:
  tracing:
    trace_config:
      service_name: 'router'
      service_namespace: 'apollo'
    datadog:
      # localhost: http://host.docker.internal:8126
      # ECS: default
      endpoint: ${env.DATADOG_AGENT_HOST:-default}
rhai:
  scripts: './rhai'

To reproduce, just run rover dev pointed to a router.yaml where path is configured to /graphql and you should see that the supergraph endpoint remains localhost:3000.

Expected result

I would have expected the supergraph's endpoint to be localhost:3000/graphql instead

Actual result

The supergraph endpoint produced by using rover dev remains localhost:3000 intead of localhost:3000/graphql

Environment

Run rover info and paste the results here

Rover Info:
Version: 0.12.2
Install Location: /Users/weschiang/Code/my-company/my-project/node_modules/binary-install/node_modules/.bin/rover
OS: Mac OS 12.6.0 [64-bit]
Shell: /bin/zsh

Apollo Studio Plan - Enterprise
Installation method - npm (yarn)

[EverlastingBugstopper]

Thanks for the report @westhechiang

@lennyburdette narrowed it down to this line where we stomp on your supergraph properties. I'll take a look at fixing this up.