bug fixes and prompt optimisation

Author: SamSaffron

assets/javascripts/discourse/components/ai-artifact.gjs

@@ -37,15 +37,14 @@ export default class AiArtifactComponent extends Component {
   }
 
   get artifactUrl() {
-    const url = getURL(
+    let url = getURL(
       `/discourse-ai/ai-bot/artifacts/${this.args.artifactId}`
     );
 
     if (this.args.artifactVersion) {
-      return `${url}?version=${this.args.artifactVersion}`;
-    } else {
-      return url;
+      url = `${url}/${this.args.artifactVersion}`;
     }
+    return url;
   }
 
   @action

lib/ai_bot/tools/update_artifact.rb

@@ -51,34 +51,26 @@ def self.signature
           {
             name: "update_artifact",
             description:
-              "Updates an existing web artifact with new HTML, CSS, or JavaScript content.",
+            "Updates an existing web artifact with new HTML, CSS, or JavaScript content. Note either html, css, or js MUST be provided. You may provide all three if desired.",
             parameters: [
               {
                 name: "artifact_id",
                 description: "The ID of the artifact to update",
                 type: "integer",
                 required: true,
               },
-              {
-                name: "html",
-                description: "(Optional) new HTML content for the artifact",
-                type: "string",
-              },
-              {
-                name: "css",
-                description: "(Optional) new CSS content for the artifact",
-                type: "string",
-              },
+              { name: "html", description: "new HTML content for the artifact", type: "string" },
+              { name: "css", description: "new CSS content for the artifact", type: "string" },
               {
                 name: "js",
-                description: "(Optional) new JavaScript content for the artifact",
+                description: "new JavaScript content for the artifact",
                 type: "string",
               },
               {
                 name: "change_description",
-                description: "A brief description of the changes being made",
+                description:
+                  "A brief description of the changes being made. Note: This only documents the change - you must provide the actual content in html/css/js parameters to make changes.",
                 type: "string",
-                required: true,
               },
             ],
           }
@@ -122,7 +114,7 @@ def invoke
                 html: parameters[:html] || last_version&.html || artifact.html,
                 css: parameters[:css] || last_version&.css || artifact.css,
                 js: parameters[:js] || last_version&.js || artifact.js,
-                change_description: parameters[:change_description],
+                change_description: parameters[:change_description].to_s,
               )
 
             update_custom_html(artifact, version)