docs: 📝 Clarify contracts and adapters#10
Conversation
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 2c8c74b4b8
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
|
|
||
| ```ts | ||
| await kernel.dependencyInjection({ | ||
| containerBuild: process.env.NODE_ENV === 'production', |
There was a problem hiding this comment.
Keep containerBuild disabled in production
This example inverts the intended build/load flow: containerBuild: true regenerates services.yaml from sourceDirectory, while false loads the generated YAML, and the existing DI guide uses NODE_ENV !== 'production'. In production deployments that ship dist plus the generated YAML but not the TS src tree, following this snippet will try to scan/rebuild at startup and fail instead of loading the container.
Useful? React with 👍 / 👎.
Summary
Validation
yarn format:checkyarn lintyarn typecheckyarn test:coverage(100%)yarn docs:buildyarn --cwd example typecheckyarn build