-
Notifications
You must be signed in to change notification settings - Fork 166
[INTEG-3264] Google docs backend can create entries #10279
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. |
| 4. Create properly formatted entries that are ready to be created in Contentful via the CMA API | ||
| Important guidelines: | ||
| CRITICAL FIELD TYPE RULES - READ CAREFULLY: |
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.
"read" 🤖
| - Text: Long text (any length) - use for descriptions, content ✓ | ||
| - Number: Integer or decimal values only ✓ | ||
| - Boolean: true or false only ✓ | ||
| - Date: ISO 8601 format (YYYY-MM-DD or YYYY-MM-DDTHH:mm:ss.sssZ) ✓ | ||
| - Location: { lat: number, lon: number } ✓ | ||
| - Object: JSON object (use sparingly, check validations) ✓ | ||
| - Array (of Symbol/Text/Number): Array of PRIMITIVE values ONLY ✓ | ||
| Example: ["value1", "value2"] or [1, 2, 3] | ||
| - Array (of Link): ❌ NEVER USE - these reference other entries, skip entirely | ||
| Example: DO NOT create [{ title: "x", content: "y" }] - this will FAIL | ||
| - Link/Reference: ❌ NEVER USE - skip these fields (they reference other entries) | ||
| - RichText: ❌ NEVER USE - complex format not supported | ||
| FIELD FORMAT RULES: |
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.
let's try one shot prompting
1464fd4 to
bbe3d5b
Compare
| - NEVER create arrays of objects like [{ title: "x", content: "y" }] - this will FAIL | ||
| - If a field type is unclear or complex, SKIP it rather than guess | ||
| COMMON MISTAKES TO AVOID: |
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.
think we need any locale comments in here? seems like a common gotcha
| // INTEG-3261: Pass the ai document response to the observer for analysis | ||
| // createDocumentObservationsFromLLMResponse() | ||
|
|
||
| // INTEG-3264: Create the entries in Contentful using the entry service |
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.
probably good to yank jira references as we complete these
| // INTEG-3264: Create the entries in Contentful using the entry service |
| // Optionally publish the entry immediately | ||
| // const publishedEntry = await cma.entry.publish( | ||
| // { spaceId, environmentId, entryId: createdEntry.sys.id }, | ||
| // createdEntry | ||
| // ); |
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.
| // Optionally publish the entry immediately | |
| // const publishedEntry = await cma.entry.publish( | |
| // { spaceId, environmentId, entryId: createdEntry.sys.id }, | |
| // createdEntry | |
| // ); |
💨
Purpose
Follow ups
Results