moved feedback prompt to txt

Author: shivakumar-ashwin

skills/fw-publish/SKILL.md

@@ -82,14 +82,9 @@ Run `cd <app-directory> && fdk validate` and treat the result as the **validity
 
 **You MUST ask this question before proceeding to fdk pack. Do not skip this step.**
 
-Ask the developer once:
+Ask the developer once — show [`references/templates/developer-feedback-rating-prompt.txt`](references/templates/developer-feedback-rating-prompt.txt) **verbatim**.
 
-> "Before we publish — how was your experience building this app with the Freshworks AI toolkit?
-> 👍 Liked it / 👎 Didn't like it / Skip"
-
-If they respond with 👍 or 👎, follow up with:
-
-> "Any specific feedback, improvements, or new features you'd like? (press Enter to skip)"
+If they respond with 👍 or 👎, follow up once — show [`references/templates/developer-feedback-comment-prompt.txt`](references/templates/developer-feedback-comment-prompt.txt) **verbatim**.
 
 Then write feedback **before proceeding to step 5** using `meta-feedback.sh` (never write JSON by hand). If the developer skips or does not respond, **do not call** `meta-feedback.sh` — omit the `"feedback"` key entirely. Never mention `.meta.json` to the developer.
 

skills/fw-publish/references/templates/developer-feedback-comment-prompt.txt

@@ -0,0 +1 @@
+Any specific feedback, improvements, or new features you'd like? (press Enter to skip)

skills/fw-publish/references/templates/developer-feedback-rating-prompt.txt

@@ -0,0 +1,2 @@
+Before we publish — how was your experience building this app with the Freshworks AI toolkit?
+👍 Liked it / 👎 Didn't like it / Skip

tests/run-inline-eval.mjs

@@ -235,7 +235,7 @@ const results = [];
 {
   const c = await skill('fw-publish');
   results.push(
-    /feedback/i.test(c) && /meta-feedback\.sh/i.test(c) && /skip/i.test(c)
+    /feedback/i.test(c) && /meta-feedback\.sh/i.test(c) && /developer-feedback-rating-prompt/i.test(c) && /skip/i.test(c)
       ? pass('fw-publish-08', 'fw-publish', 'feedback step: must ask before step 5, skip gracefully if no answer — never write null or empty')
       : fail('fw-publish-08', 'fw-publish', 'feedback step: must ask before step 5, skip gracefully if no answer — never write null or empty', 'missing feedback step or meta-feedback.sh'),
   );

tests/skill-static.test.js

@@ -352,8 +352,10 @@ describe('fw-publish SKILL.md', () => {
   test('contains feedback prompt step before fdk pack', async () => {
     const content = await readSkill('fw-publish');
     assert.ok(
-      content.includes('feedback') && (content.includes('4.5') || content.includes('Liked')),
-      'fw-publish must prompt for developer feedback before pack step'
+      content.includes('4.5')
+        && /developer-feedback-rating-prompt/i.test(content)
+        && /developer-feedback-comment-prompt/i.test(content),
+      'fw-publish must reference developer feedback templates before pack step'
     );
   });
 
@@ -570,6 +572,8 @@ describe('PR#21 — fw-publish reference files', () => {
     `${FW_PUB_REF}/templates/claude-mcp-setup.json`,
     `${FW_PUB_REF}/templates/cursor-mcp-setup.json`,
     `${FW_PUB_REF}/templates/custom-app-limit-warning.txt`,
+    `${FW_PUB_REF}/templates/developer-feedback-comment-prompt.txt`,
+    `${FW_PUB_REF}/templates/developer-feedback-rating-prompt.txt`,
     `${FW_PUB_REF}/templates/downgrade-warning.txt`,
     `${FW_PUB_REF}/templates/engines-mismatch-prompt.txt`,
     `${FW_PUB_REF}/templates/stuck-version-warning.txt`,
@@ -589,6 +593,17 @@ describe('PR#21 — fw-publish reference files', () => {
     }
   });
 
+  test('3.x: developer-feedback-rating-prompt.txt has Liked/Skip options', async () => {
+    const txt = await readRepo(`${FW_PUB_REF}/templates/developer-feedback-rating-prompt.txt`);
+    assert.ok(txt.includes('Liked it'), 'rating prompt must include Liked option');
+    assert.ok(txt.includes('Skip'), 'rating prompt must include Skip option');
+  });
+
+  test('3.x: developer-feedback-comment-prompt.txt invites optional comment', async () => {
+    const txt = await readRepo(`${FW_PUB_REF}/templates/developer-feedback-comment-prompt.txt`);
+    assert.ok(txt.includes('press Enter to skip'), 'comment prompt must allow skip');
+  });
+
   test('3.x: custom-app-limit-warning.txt mentions 25 app limit', async () => {
     const txt = await readRepo(`${FW_PUB_REF}/templates/custom-app-limit-warning.txt`);
     assert.ok(txt.includes('25'), 'must mention 25 app limit');
@@ -775,6 +790,8 @@ describe('PR#21 — internal link integrity', () => {
       'skills/fw-publish/references/templates/claude-mcp-setup.json',
       'skills/fw-publish/references/templates/engines-mismatch-prompt.txt',
       'skills/fw-publish/references/templates/custom-app-limit-warning.txt',
+      'skills/fw-publish/references/templates/developer-feedback-comment-prompt.txt',
+      'skills/fw-publish/references/templates/developer-feedback-rating-prompt.txt',
       'skills/fw-publish/references/templates/stuck-version-warning.txt',
       'skills/fw-publish/references/templates/downgrade-warning.txt',
     ];