Skip to content

Conversation

@eluce2
Copy link
Collaborator

@eluce2 eluce2 commented Jan 27, 2026

Fix #107: navigation context wasn't propagated when defaultSelect was
"schema" or object because list() returned early before reaching the
navigation propagation code.

Co-Authored-By: Claude Opus 4.5 noreply@anthropic.com


Note

Fixes missing navigation context when listing from a navigated EntitySet with defaultSelect set to "schema" or a select object.

  • Updates entity-set.ts: after applying default selects, propagate navigation metadata onto the selected QueryBuilder so URLs include the parent path (e.g., /contacts/users), and keep default top(1000) behavior
  • Adds tests in navigate.test.ts for issue fmodata: navigate() doesn't include parent table in URL path #107 covering defaultSelect='schema' navigation path inclusion with and without filters
  • Minor cleanup in scripts/test-crossjoin.ts: stricter env checks, trailing slash handling, and logging tweaks
  • Adds a changeset entry for @proofkit/fmodata patch release

Written by Cursor Bugbot for commit e3469d5. This will update automatically on new commits. Configure here.

@vercel
Copy link

vercel bot commented Jan 27, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Review Updated (UTC)
proofkit-docs Ready Ready Preview Jan 27, 2026 4:18pm

Request Review

@eluce2 eluce2 marked this pull request as ready for review January 27, 2026 16:14
Copy link
Collaborator Author

eluce2 commented Jan 27, 2026

This stack of pull requests is managed by Graphite. Learn more about stacking.

…schema

Fix #107: navigation context wasn't propagated when defaultSelect was
"schema" or object because list() returned early before reaching the
navigation propagation code.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@eluce2 eluce2 force-pushed the 01-27-fix_fmodata_navigate_includes_parent_table_when_defaultselect_is_schema branch from 1568cee to e3469d5 Compare January 27, 2026 16:16
@pkg-pr-new
Copy link

pkg-pr-new bot commented Jan 27, 2026

Open in StackBlitz

@proofkit/better-auth

pnpm add https://pkg.pr.new/proofgeist/proofkit/@proofkit/better-auth@108

@proofkit/cli

pnpm add https://pkg.pr.new/proofgeist/proofkit/@proofkit/cli@108

create-proofkit

pnpm add https://pkg.pr.new/proofgeist/proofkit/create-proofkit@108

@proofkit/fmdapi

pnpm add https://pkg.pr.new/proofgeist/proofkit/@proofkit/fmdapi@108

@proofkit/fmodata

pnpm add https://pkg.pr.new/proofgeist/proofkit/@proofkit/fmodata@108

@proofkit/typegen

pnpm add https://pkg.pr.new/proofgeist/proofkit/@proofkit/typegen@108

@proofkit/webviewer

pnpm add https://pkg.pr.new/proofgeist/proofkit/@proofkit/webviewer@108

commit: e3469d5

@eluce2 eluce2 merged commit b727425 into main Jan 27, 2026
13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

fmodata: navigate() doesn't include parent table in URL path

2 participants