-
Notifications
You must be signed in to change notification settings - Fork 2.1k
feat(ph-ai): agent artifacts schema #41834
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
Conversation
Wiz Scan Summary
To detect these findings earlier in the dev lifecycle, try using Wiz Code VS Code Extension. |
5f3aefb to
f57a488
Compare
7d6d0cb to
d153097
Compare
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.
5 files reviewed, 1 comment
f57a488 to
c22ef9d
Compare
ebe095b to
1550750
Compare
ebe095b to
4a0798a
Compare
c22ef9d to
2f258be
Compare
|
Size Change: 0 B Total Size: 3.4 MB ℹ️ View Unchanged
|
2f258be to
b1a4dd3
Compare
4a0798a to
9563d93
Compare
skoob13
left a comment
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.
Left a comment to reduce the schema bloat, as the DevEx team is trying to optimize it.
| export enum AgentArtifactType { | ||
| VISUALIZATION = 'visualization', | ||
| DOCUMENT = 'document', | ||
| } | ||
|
|
||
| export interface AgentArtifact { | ||
| id: string | ||
| short_id: string | ||
| name: string | ||
| type: AgentArtifactType | ||
| content: AgentArtifactContent | ||
| conversation: string | ||
| team: number | ||
| created_at: string | ||
| } | ||
|
|
||
| export interface AgentArtifactMinimal { | ||
| short_id: string | ||
| name: string | ||
| type: AgentArtifactType | ||
| created_at: string | ||
| } |
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.
These are types rather than schema elements. I think we just need to keep AgentArtifactContent here. In Django, it'll be a model; in TypeScript, it'll be a regular interface.
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.
Moved to types, good point
9563d93 to
f5b979d
Compare
b1a4dd3 to
23c326b
Compare
f5b979d to
53ca618
Compare
23c326b to
b465263
Compare
53ca618 to
dce6db2
Compare
b465263 to
7031a64
Compare
99d99bb to
50112ee
Compare
c926489 to
402dc8b
Compare
50112ee to
b935249
Compare
402dc8b to
fc07aea
Compare
b935249 to
cf07cd8
Compare
fc07aea to
84da926
Compare
bf9dd24 to
81d72be
Compare
514a053 to
e40b7da
Compare
81d72be to
d404f4f
Compare
e40b7da to
e096ffb
Compare
d404f4f to
66d4022
Compare
e096ffb to
3120090
Compare
ba37c70 to
a35dd23
Compare
2101098 to
3f2b627
Compare
a35dd23 to
ed59080
Compare
3f2b627 to
892a55c
Compare
ed59080 to
fa3329f
Compare
fa3329f to
46bd64f
Compare

Problem
This PR adds schema definitions for agent artifacts. These artifacts can be visualizations or documents containing markdown, visualization references, and session replay links.
Changes
frontend/src/queries/schema/schema-assistant-artifacts.ts, and added these schemas to the typegenHow did you test this code?
Added tests for artifact schemas in
ee/hogai/core/test/test_artifacts.py