This folder contains the experimental SvelteKit implementation of the Sourcegraph app.
NOTE: This is a very early prototype and it will change a lot.
# Install dependencies
pnpm install
# Run dev server
pnpm run devYou can also build the dotcom version by running pnpm run dev:dotcom, but it doesn't really differ
in functionality yet.
The dev server can be accessed on http://localhost:5173. API requests and
signin/signout are proxied to an actual Sourcegraph instance,
https://sourcegraph.com by default (can be overwritten via the
SOURCEGRAPH_API_URL environment variable.
There are some things to consider when using code from other @sourcegraph
packages:
- Since we use the barrel style of organizing our modules, many (unused) dependencies are imported into the app. This isn't ideal and at best will only increase the initial loading time. Some modules, especially those that access browser specific features during module initialization, can even cause the dev build to fail.
- Reusing code is great, but also potentially exposes someone who modifies the
reused code to this package and therefore Svelte (if the reused code changes
in an incompatible way, this package needs to be updated too). To limit the
exposure, a module of any
@sourcegraph/*package should only be imported once into this package and only into a TypeScript file. The current convention is to import any modules from@sourcegraph/commonintosrc/lib/common.ts, etc.
There are no tests yet. It would be great to try out Playwright but it looks like this depends on getting the production build working first (see below).
This package defines its own rules for formatting (which includes support for Svelte components) and linting. The workspace rules linting and formatting commands have not been updated yet to keep this experiment contained.
Run
pnpm run lint
pnpm run formatinside this directory.
There is also the pnpm run check command which uses svelte-check to validate
TypeScript, CSS, etc in Svelte components. This currently produces many errors
because it also validates imported modules from other packages, and we are not
explicitly marking type-only imports with type in other parts of the code
base (which is required by this package).
This project makes use of query composition, i.e. components define their own data dependencies via fragments, which get composed by their callers and are eventually being used in a query in a loader.
This goal of this approach is to make data dependencies co-located and easier to change, as well to make the flow of data clearer. Data fetching should only happen in data loaders, not components.
There are a couple of issues to consider with this approach and sometimes we'll have to make exceptions:
- Caching: If every loader composes its own query it's possible that two queries fetch the same data, in which case we miss out on caching. If caching the data is more important than data co-location it might be preferable to define a reusable query function. Example: File list for currently opened folder (sidebar + folder page)
- Shared data from layout loaders: While it's very convenient that pages have
access to any data from the ancestor layout loaders, that doesn't work well
with data dependency co-location. The layout loaders don't know which
sub-layout or sub-page is loaded and what data it needs.
Fortunately we don't have a lot of data (yet) that is used this way. The
prime example for this right now is information about the authenticated user.
The current approach is to name data-dependencies on the current user as
<ComponentName>_AuthenticatedUserand use that fragment in theAuthenticatedUserfragment insrc/routes/layout.gql. This approach might change as we uncover more use cases. - On demand data loading: Not all data is fetched/needed immediately for rendering page. Data for e.g. typeaheads is fetched on demand. Ideally the related queries are still composed by the data loader, which passes a function for fetching the data to the page.
A production version of this app can be built with
pnpm run buildCurrently SvelteKit is configured to create a client-side single page application.