Repository navigation
feat: in-app tracesift explore view with selected frames Investigation - #18
Merged
Merged
Conversation
There was a problem hiding this comment.
🟡 Changes recommended
A critical machine-specific dependency and multiple unresolved runtime, fallback, timeout, and state-handling issues remain.
6 open issues
1 file not reviewed
spike/package-lock.json: Generated file
What changed in this PR
Adds an embedded TraceSift Explore experience with payload transport, investigation workflows, selection handoffs, telemetry, styling, and build integration.
Changes:
- Adds task and React explorers with fullscreen and browser fallback support.
- Adds compressed MCP payload transport, client integration, and telemetry.
- Adds selection-based investigation workflows and extensive tests.
- Updates CSS/build tooling, documentation, and validation spikes.
| File | Reviewed change |
|---|---|
tracesift-embed-plan.md |
Embedding implementation plan |
tests/tracesift-ui.test.ts |
Explorer navigation and selection tests |
tests/tracesift-tools.test.ts |
MCP payload transport tests |
tests/tracesift-telemetry.test.ts |
Payload telemetry tests |
tests/tracesift-selection.test.ts |
CPU selection handoff tests |
tests/tracesift-react-selection.test.ts |
React selection handoff tests |
tests/tracesift-explore-css.test.ts |
CSS scoping tests |
tests/tracesift-client.test.ts |
Explore API client tests |
tests/tracesift-cards.test.ts |
Embedded explorer tests |
src/ui/tracesift-controller.ts |
Explorer state, navigation, selection, and telemetry |
src/ui/style.css |
Explorer and selection styling |
src/ui/components/tracesift-explorer.tsx |
Embedded explorer UI |
src/ui/components/tracesift-cards.tsx |
Cards-to-explorer switching |
src/shared/tracesift.ts |
Explore handoff types |
src/shared/tracesift-selection.ts |
CPU selection handoff |
src/shared/tracesift-react-selection.ts |
React selection handoff |
src/shared/tracesift-explore.ts |
Payload envelope and decoding |
src/server/tracesift-tools.ts |
MCP payload tool and compression |
src/server/tracesift-client.ts |
TraceSift integration clients |
src/server/telemetry.ts |
Payload telemetry |
spike/validate-ui-build.mjs |
UI build validation |
spike/package.json |
Spike dependencies |
spike/gen.mjs |
Payload generators |
spike/FINDINGS.md |
Spike findings |
spike/explore-visual/harness.tsx |
Visual explorer harness |
spike/explore-visual/fixture.ts |
Visual fixture data |
spike/explore-visual/build.mjs |
Visual build script |
spike/css/build-spike.mjs |
CSS integration spike |
spike/bridge.mjs |
Transport bridge spike |
spike/.gitignore |
Spike output exclusions |
skills/mobile-dev/SKILL.md |
Updated Explore guidance |
scripts/shared-node-modules.mjs |
Shared dependency resolution |
scripts/explore-css.mjs |
Explore stylesheet processing |
scripts/dev-ui.mjs |
Development UI integration |
scripts/build.mjs |
Production UI integration |
package.json |
Explore dependencies |
package-lock.json |
Dependency lock updates |
docs/telemetry.md |
Telemetry documentation |
docs/architecture.md |
Embedded explorer architecture |
Review effort: Lite
hurali97
changed the title
hurali97
marked this pull request as ready for review
October 21, 2026 10:20
hurali97
changed the title
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
This adds the support for in-app Profile exploration while still allowing to open in browser window. This PR also adds support to select frame(s) and investigate those explicitly.