Add comment to content

packages/database/src/queries/reports/append-report-content.ts

@@ -26,6 +26,7 @@ export const appendReportContent = async (
     const result = await db
       .update(reportFiles)
       .set({
+        // From Nate: I was just thinking we do not KNOW if this is valid markdown...
         content: sql`${reportFiles.content} || ${content}`,
         updatedAt: new Date().toISOString(),
       })

packages/database/src/queries/reports/replace-report-content.ts

@@ -20,6 +20,7 @@ type ReplaceReportContentInput = z.infer<typeof ReplaceReportContentInputSchema>
 export const replaceReportContent = async (
   params: ReplaceReportContentInput
 ): Promise<{ content: string }> => {
+  // From Nate: I was just thinking we do not KNOW if this is valid markdown...
   const { reportId, findString, replaceString } = ReplaceReportContentInputSchema.parse(params);
 
   try {

packages/database/src/queries/reports/update-report.ts

@@ -64,6 +64,7 @@ export const updateReport = async (
     }
 
     if (content !== undefined) {
+      // From Nate: I was just thinking we do not KNOW if this is valid markdown... Maybe we should accept ReportElements and then convert to markdown? Leaving this comment here for now because we are moving fast and I don't want to block the PR.
       updateData.content = content;
     }