Remove duplicate TLS helpers and consolidate DIFC/proxy utility functions

[Copilot]

A recent refactor introduced duplicate TLS helper declarations in internal/httputil, which broke package compilation. This change removes the duplicate pair and folds two small utility refactors into their existing semantic homes to reduce local duplication.

• Restore single TLS helper source in httputil

  • Remove the duplicate tls_config.go / tls_config_test.go pair.
  • Keep tls.go / tls_test.go as the canonical TLS config helpers for the package.

• Collapse DIFC tag string conversion duplication

  • Replace the private sorted tagsToStrings helper in internal/difc/reflect.go with:
    • TagsToStrings(...)
    • explicit sort.Strings(...) at the reflect-response call site
  • Drop the now-redundant helper-specific test coverage.

• Co-locate proxy arg parsing utilities

  • Move extractOwnerRepoNumber(...) from internal/proxy/proxy.go to internal/proxy/router.go, alongside the existing route arg builder/extractor helpers.
  • Keep behavior and signature unchanged.

secrecy := TagsToStrings(agent.GetSecrecyTags())
sort.Strings(secrecy)

integrity := TagsToStrings(agent.GetIntegrityTags())
sort.Strings(integrity)

agents[agentID] = ReflectedAgentLabels{
    Secrecy:   secrecy,
    Integrity: integrity,
}

[Copilot]

Pull request overview

This PR removes a duplicate TLS helper implementation that broke internal/httputil compilation, and consolidates small utility helpers to reduce local duplication in DIFC reflection and proxy tool argument parsing.

Changes:

• Delete the duplicate internal/httputil/tls_config*.go pair, keeping tls.go / tls_test.go as the single TLS helper source.
• Replace the private, sorting tagsToStrings helper with TagsToStrings plus explicit sorting at the reflect-response call site.
• Move extractOwnerRepoNumber(...) from internal/proxy/proxy.go to internal/proxy/router.go to co-locate route/arg helpers.

Show a summary per file

File	Description
internal/proxy/router.go	Adds extractOwnerRepoNumber helper alongside other route/arg utilities.
internal/proxy/proxy.go	Removes the duplicate extractOwnerRepoNumber implementation and related import.
internal/httputil/tls_config.go	Deletes duplicate TLS helper implementation (source of package compilation conflict).
internal/httputil/tls_config_test.go	Deletes duplicate TLS helper tests (canonical tests remain in tls_test.go).
internal/difc/reflect.go	Uses TagsToStrings and sorts at call site; removes private sorted helper.
internal/difc/reflect_test.go	Drops tests that directly targeted the removed private helper.

Copilot's findings

Tip

Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

• Files reviewed: 6/6 changed files
• Comments generated: 1