Skip to content

Commit

Permalink
API Update API to reflect changes to CLI interaction
Browse files Browse the repository at this point in the history
  • Loading branch information
GuySartorelli committed Sep 10, 2024
1 parent ef4cbfe commit 571d125
Show file tree
Hide file tree
Showing 16 changed files with 161 additions and 246 deletions.
17 changes: 4 additions & 13 deletions _config/dev.yml
Original file line number Diff line number Diff line change
@@ -1,19 +1,10 @@
---
Name: graphql-dev
---
SilverStripe\ORM\DatabaseAdmin:
SilverStripe\Dev\Command\DbBuild:
extensions:
- SilverStripe\GraphQL\Extensions\DevBuildExtension
- SilverStripe\GraphQL\Extensions\DbBuildExtension

SilverStripe\Dev\DevelopmentAdmin:
registered_controllers:
graphql:
controller: SilverStripe\GraphQL\Dev\DevelopmentAdmin
links:
graphql: 'List GraphQL development tools'
SilverStripe\GraphQL\Dev\DevelopmentAdmin:
registered_controllers:
build:
controller: SilverStripe\GraphQL\Dev\Build
links:
build: Build the GraphQL schema
commands:
'graphql/build': 'SilverStripe\GraphQL\Dev\SchemaBuild'
2 changes: 1 addition & 1 deletion _config/logging.yml
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@ after: '#logging'
---

SilverStripe\Core\Injector\Injector:
# Omits the HTTPOutputHandler from the logger so errors won't appear in output
# Omits the ErrorOutputHandler from the logger so errors won't appear in output
Psr\Log\LoggerInterface.graphql-quiet:
type: singleton
class: Monolog\Logger
Expand Down
24 changes: 12 additions & 12 deletions docs/en/01_getting_started/04_building_the_schema.md
Original file line number Diff line number Diff line change
Expand Up @@ -22,19 +22,19 @@ whenever the schema definition changes, or a new schema definition is added.

### What triggers a GraphQL code build?

- Any time you run the `dev/graphql/build` command to explicitly build your GraphQL schemas.
- Any time you run the `dev/build` command on your project.
- Any time you run the `sake graphql:build` command to explicitly build your GraphQL schemas.
- Any time you run the `sake db:build` command on your project.
- `silverstripe/graphql` will attempt to generate your schema "on-demand" on the first GraphQL request *only* if it wasn’t already generated.

> [!WARNING]
> Relying on the "on-demand" schema generation on the first GraphQL request requires some additional consideration.
> See [deploying the schema](06_deploying_the_schema.md#on-demand).
#### Running `dev/graphql/build`
#### Running `sake graphql:build`

The main command for generating the schema code is `dev/graphql/build`.
The main command for generating the schema code is `sake graphql:build`.

`vendor/bin/sake dev/graphql/build`
`vendor/bin/sake graphql:build`

This command takes an optional `schema` parameter. If you only want to generate a specific schema
(e.g. generate your custom schema, but not the CMS schema), you should pass in the name of the
Expand All @@ -43,19 +43,19 @@ schema you want to build.
> [!NOTE]
> If you do not provide a `schema` parameter, the command will build all schemas.
`vendor/bin/sake dev/graphql/build schema=default`
`vendor/bin/sake graphql:build --schema=default`

> [!NOTE]
> Most of the time, the name of your custom schema is `default`.
Keep in mind that some of your changes will be in YAML in the `_config/` directory, which also
requires a flush.

`vendor/bin/sake dev/graphql/build schema=default flush=1`
`vendor/bin/sake graphql:build --schema=default --flush`

#### Building on dev/build
#### Building with `sake db:build`

By default, all schemas will be built during `dev/build`. To disable this, change the config:
By default, all schemas will be built during `sake db:build`. To disable this, change the config:

```yml
SilverStripe\GraphQL\Extensions\DevBuildExtension:
Expand All @@ -74,11 +74,11 @@ If the type hasn't changed, it doesn't get re-built. This reduces build times to

#### Clearing the schema cache

If you want to completely re-generate your schema from scratch, you can add `clear=1` to the `dev/graphql/build` command.
If you want to completely re-generate your schema from scratch, you can add `--clear` to the `sake graphql:build` command.

`vendor/bin/sake dev/graphql/build schema=default clear=1`
`vendor/bin/sake graphql:build --schema=default --clear`

If your schema is producing unexpected results, try using `clear=1` to eliminate the possibility
If your schema is producing unexpected results, try using `--clear` to eliminate the possibility
of a caching issue. If the issue is resolved, record exactly what you changed and [create an issue](https://github.com/silverstripe/silverstripe-graphql/issues/new).

### Build gotchas
Expand Down
16 changes: 8 additions & 8 deletions docs/en/01_getting_started/06_deploying_the_schema.md
Original file line number Diff line number Diff line change
Expand Up @@ -14,13 +14,13 @@ One way or another, you must get the `.graphql-generated/` and `public/_graphql/

### Single-server hosting solutions with simple deployments {#simple-single-server}

If you host your site on a single server and you always run `dev/build` during the deployment, then assuming you have set up permissions to allow the webserver to write to the `.graphql-generated/` and `public/_graphql/` folders, your GraphQL schema will be built for you as a side-effect of running `dev/build`. You don't need to do anything further. Note that if your schema is exceptionally large you may still want to read through the rest of the options below.
If you host your site on a single server and you always run `sake db:build` during the deployment, then assuming you have set up permissions to allow the webserver to write to the `.graphql-generated/` and `public/_graphql/` folders, your GraphQL schema will be built for you as a side-effect of running `sake db:build`. You don't need to do anything further. Note that if your schema is exceptionally large you may still want to read through the rest of the options below.

### Options for any hosting solution

#### Commit the schema to version control {#commit-to-vcs}

A simplistic approach is to build the schema in your local development environment and add the `.graphql-generated/` and `public/_graphql/` folders to your version control system. With this approach you would most likely want to disable schema generation at `dev/build`.
A simplistic approach is to build the schema in your local development environment and add the `.graphql-generated/` and `public/_graphql/` folders to your version control system. With this approach you would most likely want to disable schema generation with `sake db:build`.

This approach has the advantage of being very simple, but it will pollute your commits with massive diffs for the generated code.

Expand All @@ -29,19 +29,19 @@ This approach has the advantage of being very simple, but it will pollute your c
#### Explicitly build the schema during each deployment {#build-during-deployment}

Many projects will automatically run a `dev/build` whenever they deploy a site to their production environment. If that’s your case, then you can just let this process run normally and generate the `.graphql-generated/` and `public/_graphql/` folders for you. This will allow you to add these folders to your `.gitignore` file and avoid tracking the folder in your version control system.
Many projects will automatically run `sake db:build` whenever they deploy a site to their production environment. If that’s your case, then you can just let this process run normally and generate the `.graphql-generated/` and `public/_graphql/` folders for you. This will allow you to add these folders to your `.gitignore` file and avoid tracking the folder in your version control system.

Be aware that for this approach to work, the process executing the `dev/build` must have write access to create the folders (or you must create those folders yourself, and give write access for those folders specifically), and for multi-server environments a `dev/build` or `dev/graphql/build` must be executed on each server hosting your site after each deployment.
Be aware that for this approach to work, the process executing `sake db:build` must have write access to create the folders (or you must create those folders yourself, and give write access for those folders specifically), and for multi-server environments `sake db:build` or `sake graphql:build` must be executed on each server hosting your site after each deployment.

#### Use a CI/CD pipeline to build your schema {#using-ci-cd}

Projects with more sophisticated requirements or bigger schemas exposing more than 100 `DataObject` classes may want to consider using a continuous-integration/continuous-deployment (CI/CD) pipeline to build their GraphQL schema.

In this kind of setup, you would need to update your deployment script to run the `dev/graphql/build` command which builds the `.graphql-generated/` and `public/_graphql/` folders. In multi-server environments this must be executed on each server hosting your site.
In this kind of setup, you would need to update your deployment script to run the `sake graphql:build` command which builds the `.graphql-generated/` and `public/_graphql/` folders. In multi-server environments this must be executed on each server hosting your site.

### Multi-server hosting solutions {#multi-server}

If your site is hosted in an environment with multiple servers or configured to auto-scale with demand, there are some additional considerations. For example if you only generate the schema on one single server (i.e. via `dev/build` or `dev/graphql/build`), then the other servers won’t have a `.graphql-generated/` or `public/_graphql/` folder (or those folders will be empty if you manually created them).
If your site is hosted in an environment with multiple servers or configured to auto-scale with demand, there are some additional considerations. For example if you only generate the schema on one single server (i.e. via `sake db:build` or `sake graphql:build`), then the other servers won’t have a `.graphql-generated/` or `public/_graphql/` folder (or those folders will be empty if you manually created them).

#### Rely on "on-demand" schema generation on the first GraphQL request {#on-demand}

Expand All @@ -58,7 +58,7 @@ Our expectation is that on-demand schema generation will be performant for most
#### Build the schema during/before deployment and share it across your servers {#multi-server-shared-dirs}

If you have a particularly large schema, you may want to ensure it is always built before the first GraphQL request. It might make sense for you to sync your `.graphql-generated/` and `public/_graphql/` folders across all your servers using an EFS or similar mechanism. In that case you only need to run `dev/build` or `dev/graphql/build` on the server with the original folder - but bear in mind that this may have a performance impact.
If you have a particularly large schema, you may want to ensure it is always built before the first GraphQL request. It might make sense for you to sync your `.graphql-generated/` and `public/_graphql/` folders across all your servers using an EFS or similar mechanism. In that case you only need to run `sake db:build` or `sake graphql:build` on the server with the original folder - but bear in mind that this may have a performance impact.

### Performance considerations when building the GraphQL schema {#performance-considerations}

Expand All @@ -81,7 +81,7 @@ DataObjects in schema | Build time (ms) | Memory use (MB)

The process that is generating these folders must have write permissions to create the folder and to update existing files. If different users are used to generate the folders, then you must make sure that each user retains write access on them.

For example, if you manually run a `dev/build` under a foobar user, the folders will be owned by foobar. If your web server is running under the www-data user and you try to call `dev/graphql/build` in your browser, you might get an error if www-data doesn’t have write access.
For example, if you manually run `sake db:build` under a foobar user, the folders will be owned by foobar. If your web server is running under the www-data user and you try to visit `dev/graphql/build` in your browser, you might get an error if www-data doesn’t have write access.

### Further reading

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -56,9 +56,9 @@ The `*` value on `operations` tells the schema to create all available queries a
- `update`
- `delete`

Now that we've changed our schema, we need to build it using the `dev/graphql/build` command:
Now that we've changed our schema, we need to build it using the `graphql:build` command:

`vendor/bin/sake dev/graphql/build schema=default`
`vendor/bin/sake graphql:build --schema=default`

Now we can access our schema on the default GraphQL endpoint, `/graphql`.

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -67,7 +67,7 @@ queries:

Now, we just have to build the schema:

`vendor/bin/sake dev/graphql/build schema=default`
`vendor/bin/sake graphql:build --schema=default`

### Testing the query

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -145,7 +145,7 @@ resolvers:

Re-run the schema build, with a flush (because we created a new PHP class), and let's go!

`vendor/bin/sake dev/graphql/build schema=default flush=1`
`vendor/bin/sake graphql:build --schema=default flush=1`

### Field resolvers

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -96,7 +96,7 @@ can handle the rest. It will return an array including `edges`, `nodes`, and `pa

Rebuild the schema and test it out:

`vendor/bin/sake dev/graphql/build schema=default`
`vendor/bin/sake graphql:build --schema=default`

```graphql
query {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@ summary: Add descriptions to just about anything in your schema to improve your

One of the great features of a schema-backed API is that it is self-documenting. If you use
the [`silverstripe/graphql-devtools`](https://github.com/silverstripe/silverstripe-graphql-devtools)
module you can see the documentation by navigating to /dev/graphql/ide in your browser anc clicking
module you can see the documentation by navigating to `/dev/graphql/ide` in your browser anc clicking
on "DOCS" on the right.

Many API developers choose to maximise the benefit of this by adding descriptions to some or
Expand Down
2 changes: 1 addition & 1 deletion docs/en/07_tips_and_tricks.md
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@ to your environment file. The classnames and filenames in the generated code dir

> [!WARNING]
> Take care not to use `DEBUG_SCHEMA=1` as an inline environment variable to your build command, e.g.
> `DEBUG_SCHEMA=1 vendor/bin/sake dev/graphql/build` because any activity that happens at run time, e.g. querying the schema
> `DEBUG_SCHEMA=1 vendor/bin/sake graphql:build` because any activity that happens at run time, e.g. querying the schema
> will fail, since the environment variable is no longer set.
In live mode, full obfuscation kicks in and the filenames become unreadable. You can only determine the type they map
Expand Down
2 changes: 1 addition & 1 deletion docs/en/08_architecture_diagrams.md
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,7 @@ The schema is generated during a build step, which generates code generation art

![A high-level view of the GraphQL build process](_images/build_process.png)

- **`/dev/graphql/build`**: This is the command that builds the schema. It also runs as a side effect of `dev/build` as a fallback. It accepts a `schema` parameter if you only want to build one schema.
- **`sake graphql:build`**: This is the command that builds the schema. It also runs as a side effect of `sake db:build` as a fallback. It accepts a `schema` parameter if you only want to build one schema.

- **Schema Factory**: This class is responsible for rebuilding a schema or fetching an existing one (i.e. as cached generated code)

Expand Down
146 changes: 0 additions & 146 deletions src/Dev/DevelopmentAdmin.php

This file was deleted.

Loading

0 comments on commit 571d125

Please sign in to comment.