You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
Repository navigation
docs(experimentation): Point the coding agent prompts at the Flagsmith database - #8892
The coding agent prompts still expect the Kafka stack that #8884 removed from docker compose, so they would fail on the new compose file.
Local setup prompt: drop port 8080, kafka-init and clickhouse-init; connect environments to the Flagsmith database instead of the bundled ClickHouse.
Experiment set-up prompt: send SDK events to http://localhost:8000/api/ instead of the ingestion service.
Say the Flagsmith database is for trying out experimentation.
How did you test this code?
Ran the warehouse connection call from the updated prompt against the new compose stack; it returned connected. Events sent to http://localhost:8000/api/v1/events landed and showed in experiment results.
The coding-agent guides now use the local API on port 8000 for SDK events. The local setup guide checks ports 8000 and 8001, waits for the API liveness endpoint and successful database migration, and connects experimentation environments to the Flagsmith PostgreSQL database. It removes the event-ingestion service from the service summary and states that the database is intended only for experimentation trials.
Existing local experimentation setups can stop receiving events when following the updated instructions. Clarify the SDK address and handle older stacks and ClickHouse connections before merging.
- [ ] Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 2
Note
Follow-up findings are listed below so you can review them separately from inline comments.
Follow-up (1)
Major (1)
Major · Check the warehouse before selecting the local events URL. · set-up-with-a-coding-agent.mdx:98-99
docs/docs/experimentation/set-up-with-a-coding-agent.mdx:98-99 Major · Functional Correctness · Quick win
Check the warehouse before selecting the local events URL.
For an existing local ClickHouse connection, this URL makes SDK event batches return 404, so the experiment receives no new events. Direct ClickHouse users to their ingestion service, or ask them to migrate to the Flagsmith database before using this URL.
Review info
Run configuration
Source mode: Auto
Repository: Unavailable
Branch: Unavailable
File: No configuration file from this repository applied
Commit: Unavailable
Layers: Organization UI
Review profile: ASSERTIVE
Plan: Advanced
Run ID: 762c703b-aa14-45c0-99e9-d3b02202f2a8
Commits
Reviewing files that changed from the base of the PR and between 1ec1049 and 444f00e.
Included review availability: This review used your included allowance. Your plan provides up to 8 included reviews per hour; 7 remain after this review.
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
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.
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
Thanks for submitting a PR! Please check the boxes below:
docs/if required so people know about the feature.Changes
Contributes to #8884
The coding agent prompts still expect the Kafka stack that #8884 removed from docker compose, so they would fail on the new compose file.
kafka-initandclickhouse-init; connect environments to the Flagsmith database instead of the bundled ClickHouse.http://localhost:8000/api/instead of the ingestion service.How did you test this code?
Ran the warehouse connection call from the updated prompt against the new compose stack; it returned
connected. Events sent tohttp://localhost:8000/api/v1/eventslanded and showed in experiment results.