Refactor missing object routes to use BaseTableRouteHandler pattern

[Copilot]

The /objects/missing routes were manually mapped in ObjectRouteHandler.MapAdditionalRoutes() instead of following the ITableRouteHandler pattern used by other route handlers.

Changes

• Created ObjectMissingRouteHandler

  • Implements ITableRouteHandler with BaseRoute = "/missing"
  • Migrated ListMissingObjects and AddMissingObject methods from ObjectRouteHandler
  • Added stub implementations for Read/Update/Delete returning 501 Not Implemented

• Updated ObjectRouteHandler.MapAdditionalRoutes

  • Replaced manual MapGet/MapPost calls with BaseTableRouteHandler.MapRoutes<ObjectMissingRouteHandler>(parentRoute)
  • Removed migrated methods

Example

Before:

public static void MapAdditionalRoutes(IEndpointRouteBuilder parentRoute)
{
    _ = parentRoute.MapGet(RoutesV2.Missing, ListMissingObjects);
    _ = parentRoute.MapPost(RoutesV2.Missing, AddMissingObject);
    // ... other routes
}

After:

public static void MapAdditionalRoutes(IEndpointRouteBuilder parentRoute)
{
    BaseTableRouteHandler.MapRoutes<ObjectMissingRouteHandler>(parentRoute);
    // ... other routes
}

API routes unchanged: GET/POST /v2/objects/missing

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

• av-build-tel-api-v1.avaloniaui.net
  • Triggering command: /usr/share/dotnet/dotnet dotnet exec --runtimeconfig /home/REDACTED/.nuget/packages/avalonia.buildservices/11.3.1/tools/netstandard2.0/runtimeconfig.json /home/REDACTED/.nuget/packages/avalonia.buildservices/11.3.1/tools/netstandard2.0/Avalonia.BuildServices.Collector.dll (dns block)
  • Triggering command: /usr/share/dotnet/dotnet dotnet exec --runtimeconfig /home/REDACTED/.nuget/packages/avalonia.buildservices/11.3.2/tools/netstandard2.0/runtimeconfig.json /home/REDACTED/.nuget/packages/avalonia.buildservices/11.3.2/tools/netstandard2.0/Avalonia.BuildServices.Collector.dll (dns block)
  • Triggering command: /usr/share/dotnet/dotnet dotnet exec --runtimeconfig /home/REDACTED/.nuget/packages/avalonia.buildservices/11.3.1/tools/netstandard2.0/runtimeconfig.json /home/REDACTED/.nuget/packages/avalonia.buildservices/11.3.1/tools/netstandard2.0/Avalonia.BuildServices.Collector.dll rs/ObjectMissingRouteHandler.cs ctService/RouteH--logger:console;verbosity=normal ctService/RouteH/home/REDACTED/work/ObjectEditor/ObjectEditor/Tests/bin/Release/net10.0/Tests.dll ctService/RouteHbash ctService/RouteH--norc ctService/RouteH--noprofile ctService/RouteHandlers/TableHan/nologo ctSe�� ctService/RouteHandlers/TableHan/nodeReuse:true /netstandard2.0/runtimeconfig.json /netstandard2.0/Avalonia.BuildServices.Collector.dll ctService/RouteHbash ctService/RouteH--norc ctService/RouteH--noprofile ctService/RouteHandlers/BaseData/nologo (dns block)

If you need me to access, download, or install something from one of these locations, you can either:

• Configure Actions setup steps to set up my environment, which run before the firewall is enabled
• Add the appropriate URLs or hosts to the custom allowlist in this repository's Copilot coding agent settings (admins only)

Original prompt

in ObjectRouteHandler.cs, there are 2 additional routes added for Missing, which will resolve to "/objects/missing". Instead of adding these manually, we should create a full route handler class for it (called "ObjectMissingRouteHandler") that implements ITableRouteHandler as per the other route handlers, and then in ObjectRouteHandler we can reuse the BaseTableRouteHandler.MapRoutes method with BaseTableRouteHandler.MapRoutes<ObjectMissingRouteHandler>(parentRoute).

---

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

[Copilot]

Pull request overview

This PR refactors the missing object routes (/v2/objects/missing) to use the ITableRouteHandler pattern instead of manual route mapping, improving consistency with other route handlers in the codebase.

Key Changes:

• Created ObjectMissingRouteHandler implementing ITableRouteHandler with full CRUD operations
• Migrated ListMissingObjects and AddMissingObject from ObjectRouteHandler to the new handler
• Added implementations for Read and Delete operations; Update returns 501 Not Implemented
• Created DtoMissingObjectEntryComparer for comparing missing object entries

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 4 comments.

File	Description
ObjectService/RouteHandlers/TableHandlers/ObjectMissingRouteHandler.cs	New route handler implementing ITableRouteHandler pattern for missing objects with List, Create, Read, Update (stub), and Delete operations
ObjectService/RouteHandlers/TableHandlers/ObjectRouteHandler.cs	Removed manual route mapping for missing objects and replaced with BaseTableRouteHandler.MapRoutes call
Definitions/DTO/Comparers/DtoMissingObjectEntryComparer.cs	New comparer for DtoMissingObjectEntry based on DatName, DatChecksum, and ObjectType
Tests/ObjectServiceIntegrationTests/ObjectMissingRoutesTest.cs	Stub test file with TODO for future test implementation