-
-
Notifications
You must be signed in to change notification settings - Fork 21.4k
UPDATE #1218
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Open
Roblmvp
wants to merge
19
commits into
f:main
Choose a base branch
from
Roblmvp:main
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
UPDATE #1218
Changes from all commits
Commits
Show all changes
19 commits
Select commit
Hold shift + click to select a range
ebbeae3
Update .gitignore
Roblmvp 1a66257
Update .gitignore
Roblmvp 2d35997
Update prompts.config.ts
Roblmvp eee0c15
Update prompts.config.ts
Roblmvp 04cfa1b
feat(db): add multi-tenancy and execution trace tables (Step 1)
claude d1f357e
feat(db): backfill default internal organization (Step 1b)
claude ad9af07
feat(observability): structured logging + request correlation (Step 2)
claude b40fe70
fix(sentry): source DSN from env and disable PII (Step 2a)
claude d865273
fix(security): revoke Data-API grants + enable RLS on public tables
claude 9fe7f47
feat(db): add Composition + Stage models for Composition Engine (Phas…
claude 3579787
feat(db): apply composition engine migration + org FK + RLS
claude fe276a4
feat(composition): Composition Engine orchestrator + API (Level 2)
claude 1cf5ee9
test(composition): add live smoke script for /execute over HTTP
claude 6516df9
fix(security): derive identity server-side in composition routes
claude 5bf493d
feat(ai): add Anthropic provider with prompt caching (default-off)
claude b08826e
docs: Level 3 blueprint — validation, governed promotion, routing, MCP
claude a655a27
fix(build): let prisma generate run without DATABASE_URL
claude de760ed
Merge pull request #1 from Roblmvp/claude/wizardly-dirac-mhzb9v
Roblmvp 4bd1ac5
Merge branch 'f:main' into main
Roblmvp File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
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
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -49,3 +49,5 @@ next-env.d.ts | |
| # Sentry Config File | ||
| .env.sentry-build-plugin | ||
| .env.docker | ||
| .env.production.vyaxis | ||
| .env* | ||
Oops, something went wrong.
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🗄️ Data Integrity & Integration | 🔴 Critical | ⚡ Quick win
.env*on line 53 shadows the!.env.examplenegation on line 37, causing.env.exampleto be ignored.In
.gitignore, the last matching pattern wins. Since.env*(line 53) comes after!.env.example(line 37), it re-ignores.env.example. This means the example env template — which should be tracked — will be excluded from git, andgit statusmay show it as deleted if already tracked.🐛 Proposed fix
Move the catch-all before the negation, or add a second negation after it:
📝 Committable suggestion
🤖 Prompt for AI Agents