githubnext/ace2Public

Ace: a collaborative coding environment on the team's own machines

Find applications omitted by bounded desktop inventory

IA

iamnbutler opened ·

During #8 / #5 dogfooding, the native bridge was healthy but `desktop_apps` omitted the newly launched Ace process (PID 56500) because the host trimmed the result to its 32 KB text limit. The returned result contained 107 apps, `ace_truncated: true`, and `ace_omitted: 42`; native `inventory_completeness` was `complete`. A readiness helper mistook the app's absence for startup failure.

Evidence: `/tmp/ace-windows-readiness-result.json`. The existing fixture remained listed, so validation could continue; this is a discovery limitation, not evidence that the native bridge failed.

Add an optional bounded, nonblank `desktop_apps` query that searches application names and bundle IDs case-insensitively in the full native result **before** host result bounding. Preserve native completeness/warnings, explain that filtering covers only that returned native inventory, and report matching/total counts. Keep the global payload cap unchanged. Do not infer that an app is absent from the machine when inventory is partial or results remain truncated.

  1. iamnbutlerreferenced this in Search desktop applications before bounding inventory results #84last month on GitHub

  2. iamnbutlerreferenced this in [Meta] Dogfood Ace end to end #5last month on GitHub

  3. iamnbutlerreferenced this in [Meta] Complete native computer use in Ace #8last month on GitHub

  4. IA
    iamnbutler
    Author

    Dogfooding found #83: a healthy native bridge's application list was trimmed to 107 apps, omitting the newly launched Ace process. https://github.com/githubnext/ace2/pull/84 adds an optional name/bundle-ID search before bounding, preserving the text cap and native completeness plus explicit filter counts. Static checks and source review pass; a read-only check on the current signed runtime is prepared. This does not block #74's fixture validation.

  5. IA
    iamnbutler
    Author

    Merged #84 as `eee1a39` after type/lint checks, Ace diff review, independent source review, and the real read-only smoke on exact head `ee71fe0`. The original omission reproduced (107 returned/42 omitted); name and bundle-ID queries found Ace's exact process generation, while no-match/invalid queries and the unchanged 32 KB cap passed. Native completeness and target metadata were preserved. Evidence: `/tmp/ace-app-search-validation-LEAHpI/result.json`. No model run or new native build was performed for this host-only behavior.

  6. iamnbutlerclosedlast month on GitHub

  7. iamnbutlerreferenced this in Land desktop inspection/actions and tab API from feat/github-cache #89last month on GitHub