auditkit 0.1.1 → 0.1.4

package/src/main.rs

@@ -1,4 +1,4 @@
-use std::io::{self, Write};
+use std::{fs, process};
 
 use anyhow::{Context, Result};
 use auditkit::audit::{slugify, split_comma_list, AuditInput};
@@ -76,28 +76,19 @@ fn run() -> Result<()> {
 }
 
 fn new_audit(workspace: &Workspace) -> Result<()> {
-    ui::section("New Audit");
-    let client_name = prompt("Client name            ")?;
-    let url = prompt("Website URL            ")?;
-    let business_type = prompt("Business type          ")?;
-    let goal = prompt("Primary goal           ")?;
-    let target_customer = prompt("Target customer        ")?;
-    let conversion_action = prompt("Main conversion action ")?;
-    let pages = prompt("Pages, comma-separated ")?;
-    let known_concerns = prompt("Known concerns         ")?;
-    let competitors = prompt("Competitors            ")?;
+    let answers = ui::collect_audit_input()?;
 
     let audit = AuditInput {
-        slug: slugify(&client_name),
-        client_name,
-        url,
-        business_type,
-        goal,
-        target_customer,
-        conversion_action,
-        pages: split_comma_list(&pages),
-        known_concerns: split_comma_list(&known_concerns),
-        competitors: split_comma_list(&competitors),
+        slug: slugify(&answers.client_name),
+        client_name: answers.client_name,
+        url: answers.url,
+        business_type: answers.business_type,
+        goal: answers.goal,
+        target_customer: answers.target_customer,
+        conversion_action: answers.conversion_action,
+        pages: split_comma_list(&answers.pages),
+        known_concerns: split_comma_list(&answers.known_concerns),
+        competitors: split_comma_list(&answers.competitors),
         created_at: Local::now().format("%Y-%m-%d").to_string(),
     };
 
@@ -105,7 +96,7 @@ fn new_audit(workspace: &Workspace) -> Result<()> {
     ui::section("Audit Created");
     ui::saved(folder.display());
     ui::bullet("Next: ak inspect latest");
-    ui::bullet("Then fill findings.md, scorecard.md, and pages/*.md");
+    ui::bullet("Then fill workspace.md and findings.md");
     Ok(())
 }
 
@@ -130,9 +121,9 @@ fn check(workspace: &Workspace, target: &str, save: Option<&str>) -> Result<()>
         println!("{}", html_check::format_cli(&result));
         if let Some(folder) = save {
             let folder = workspace.resolve_target(Some(folder))?;
-            let path = workspace.write_audit_file(
+            let path = workspace.update_workspace_section(
                 &folder,
-                "automated-check.md",
+                "Automated Check",
                 &html_check::format_markdown(&result),
             )?;
             ui::saved(path.display());
@@ -146,9 +137,9 @@ fn check(workspace: &Workspace, target: &str, save: Option<&str>) -> Result<()>
         html_check::check_url(&website)
     })?;
     println!("{}", html_check::format_cli(&result));
-    let path = workspace.write_audit_file(
+    let path = workspace.update_workspace_section(
         &folder,
-        "automated-check.md",
+        "Automated Check",
         &html_check::format_markdown(&result),
     )?;
     ui::saved(path.display());
@@ -163,9 +154,9 @@ fn security_check(workspace: &Workspace, target: Option<&str>, save: Option<&str
         println!("{}", security::format_cli(&result));
         if let Some(folder) = save {
             let folder = workspace.resolve_target(Some(folder))?;
-            let path = workspace.write_audit_file(
+            let path = workspace.update_workspace_section(
                 &folder,
-                "security.md",
+                "Security Check",
                 &security::format_markdown(&result),
             )?;
             ui::saved(path.display());
@@ -179,8 +170,11 @@ fn security_check(workspace: &Workspace, target: Option<&str>, save: Option<&str
         security::check_url(&website)
     })?;
     println!("{}", security::format_cli(&result));
-    let path =
-        workspace.write_audit_file(&folder, "security.md", &security::format_markdown(&result))?;
+    let path = workspace.update_workspace_section(
+        &folder,
+        "Security Check",
+        &security::format_markdown(&result),
+    )?;
     ui::saved(path.display());
     Ok(())
 }
@@ -189,29 +183,71 @@ fn lighthouse_check(workspace: &Workspace, target: Option<&str>, save: Option<&s
     let target = target.unwrap_or("latest");
 
     if looks_like_url(target) {
-        let output_folder = match save {
-            Some(folder) => Some(workspace.audit_folder(&workspace.resolve_target(Some(folder))?)),
-            None => None,
-        };
+        let save_folder = save
+            .map(|folder| workspace.resolve_target(Some(folder)))
+            .transpose()?;
+        let temp_dir = save_folder.as_ref().map(|_| lighthouse_temp_dir(workspace));
+        if let Some(folder) = &temp_dir {
+            fs::create_dir_all(folder)?;
+        }
         let paths = ui::with_task("Running Lighthouse in Helium", || {
-            lighthouse::run_lighthouse(&workspace.root, target, output_folder.as_deref())
+            lighthouse::run_lighthouse(&workspace.root, target, temp_dir.as_deref())
         })?;
-        ui::saved(paths.markdown_path.display());
-        ui::saved(paths.json_path.display());
+        print_lighthouse_output(&paths.cli_output);
+        if let Some(folder) = save_folder {
+            save_lighthouse_summary(workspace, &folder, &paths)?;
+        } else {
+            ui::saved(paths.markdown_path.display());
+            ui::saved(paths.json_path.display());
+        }
+        if let Some(folder) = temp_dir {
+            let _ = fs::remove_dir_all(folder);
+        }
         return Ok(());
     }
 
     let folder = workspace.resolve_target(Some(target))?;
     let website = audit_website(workspace, &folder)?;
-    let audit_folder = workspace.audit_folder(&folder);
+    let temp_dir = lighthouse_temp_dir(workspace);
+    fs::create_dir_all(&temp_dir)?;
     let paths = ui::with_task("Running Lighthouse in Helium", || {
-        lighthouse::run_lighthouse(&workspace.root, &website, Some(&audit_folder))
+        lighthouse::run_lighthouse(&workspace.root, &website, Some(&temp_dir))
     })?;
-    ui::saved(paths.markdown_path.display());
-    ui::saved(paths.json_path.display());
+    print_lighthouse_output(&paths.cli_output);
+    save_lighthouse_summary(workspace, &folder, &paths)?;
+    let _ = fs::remove_dir_all(temp_dir);
     Ok(())
 }
 
+fn print_lighthouse_output(output: &str) {
+    if !output.trim().is_empty() {
+        println!("{output}");
+    }
+}
+
+fn save_lighthouse_summary(
+    workspace: &Workspace,
+    folder: &str,
+    paths: &lighthouse::LighthousePaths,
+) -> Result<()> {
+    let markdown = fs::read_to_string(&paths.markdown_path)?;
+    let json = fs::read_to_string(&paths.json_path)?;
+    let workspace_path =
+        workspace.update_workspace_section(folder, "Lighthouse Check", &markdown)?;
+    let json_path = workspace.write_audit_file(folder, "raw/lighthouse.json", &json)?;
+    ui::saved(workspace_path.display());
+    ui::saved(json_path.display());
+    Ok(())
+}
+
+fn lighthouse_temp_dir(workspace: &Workspace) -> std::path::PathBuf {
+    workspace.root.join("target").join(format!(
+        "auditkit-lighthouse-{}-{}",
+        process::id(),
+        Local::now().timestamp_millis()
+    ))
+}
+
 fn inspect(workspace: &Workspace, target: Option<&str>) -> Result<()> {
     let folder = workspace.resolve_target(target)?;
     ui::section("Inspect");
@@ -247,11 +283,3 @@ fn audit_website(workspace: &Workspace, folder: &str) -> Result<String> {
 fn looks_like_url(value: &str) -> bool {
     value.starts_with("http://") || value.starts_with("https://") || value.contains('.')
 }
-
-fn prompt(label: &str) -> Result<String> {
-    print!("{label}");
-    io::stdout().flush()?;
-    let mut value = String::new();
-    io::stdin().read_line(&mut value)?;
-    Ok(value.trim().to_string())
-}

package/src/report.rs

@@ -100,7 +100,7 @@ pub fn build_final_report(
     files: &BTreeMap<String, String>,
 ) -> String {
     let brief = parse_brief(files.get("brief.md").map(String::as_str).unwrap_or(""));
-    let page_sections = files
+    let old_page_sections = files
         .iter()
         .filter(|(path, _)| path.starts_with("pages/") && path.ends_with(".md"))
         .map(|(path, content)| {
@@ -112,6 +112,9 @@ pub fn build_final_report(
         })
         .collect::<Vec<_>>()
         .join("\n\n");
+    let page_sections = workspace_section(files, "Page Reviews", "")
+        .filter(|content| !content.contains("No page reviews provided yet."))
+        .unwrap_or(old_page_sections);
 
     format!(
         "# {} Website Audit\n\nPrepared by {}, {}\nAudit folder: {}\n\n## Context\n\nWebsite: {}\nGoal: {}\nBusiness type: {}\nTarget customer: {}\nMain conversion action: {}\n\n## Scorecard\n\n{}\n\n## Automated Check\n\n{}\n\n## Security Check\n\n{}\n\n## Lighthouse Check\n\n{}\n\n## Findings\n\n{}\n\n## Page Reviews\n\n{}\n\n## Recommended Next Step\n\nAudit: {}\nRefresh: {}\nGrowth: {}\n\nUse the audit to prioritise immediate fixes. If most findings require design, copy, and frontend changes, the Refresh package is likely the best next step.\n",
@@ -124,10 +127,14 @@ pub fn build_final_report(
         empty_or(&brief.business_type, "Not specified"),
         empty_or(&brief.target_customer, "Not specified"),
         empty_or(&brief.conversion_action, "Not specified"),
-        section_file(files, "scorecard.md", "Not completed yet."),
-        section_file(files, "automated-check.md", "Not run yet."),
-        section_file(files, "security.md", "Not run yet."),
-        section_file(files, "lighthouse.md", "Not run yet."),
+        workspace_section(files, "Scorecard", "Not completed yet.")
+            .unwrap_or_else(|| section_file(files, "scorecard.md", "Not completed yet.")),
+        workspace_section(files, "Automated Check", "Not run yet.")
+            .unwrap_or_else(|| section_file(files, "automated-check.md", "Not run yet.")),
+        workspace_section(files, "Security Check", "Not run yet.")
+            .unwrap_or_else(|| section_file(files, "security.md", "Not run yet.")),
+        workspace_section(files, "Lighthouse Check", "Not run yet.")
+            .unwrap_or_else(|| section_file(files, "lighthouse.md", "Not run yet.")),
         section_file(files, "findings.md", "No findings captured yet."),
         if page_sections.is_empty() { "No page reviews captured yet.".to_string() } else { page_sections },
         config.audit_price,
@@ -166,6 +173,29 @@ fn section_file(files: &BTreeMap<String, String>, path: &str, fallback: &str) ->
         .unwrap_or_else(|| fallback.to_string())
 }
 
+fn workspace_section(
+    files: &BTreeMap<String, String>,
+    heading: &str,
+    fallback: &str,
+) -> Option<String> {
+    let workspace = files.get("workspace.md")?;
+    let heading = format!("## {heading}");
+    let lines = workspace.lines().collect::<Vec<_>>();
+    let start = lines.iter().position(|line| line.trim() == heading)? + 1;
+    let end = lines
+        .iter()
+        .enumerate()
+        .skip(start)
+        .find_map(|(index, line)| line.starts_with("## ").then_some(index))
+        .unwrap_or(lines.len());
+    let content = lines[start..end].join("\n").trim().to_string();
+    if content.is_empty() {
+        Some(fallback.to_string())
+    } else {
+        Some(content)
+    }
+}
+
 fn empty_or<'a>(value: &'a str, fallback: &'a str) -> &'a str {
     if value.trim().is_empty() {
         fallback
@@ -247,4 +277,27 @@ mod tests {
         assert!(report.contains("## Lighthouse Check"));
         assert!(report.contains("### Homepage Review"));
     }
+
+    #[test]
+    fn final_report_reads_consolidated_workspace_sections() {
+        let mut files = BTreeMap::new();
+        files.insert(
+            "brief.md".to_string(),
+            "# Acme Dental Audit Brief\n\nWebsite: https://example.com\nGoal: Leads".to_string(),
+        );
+        files.insert(
+            "workspace.md".to_string(),
+            "# Audit Workspace\n\n## Scorecard\n\nPerformance: 8/10\n\n## Automated Check\n\nScore: 74/100\n\n## Security Check\n\nScore: 50/100\n\n## Lighthouse Check\n\nPerformance: 87/100\n\n## Page Reviews\n\n### Homepage Review\n\nCTA needs work.\n".to_string(),
+        );
+        files.insert(
+            "findings.md".to_string(),
+            "# Findings\n\n## Critical".to_string(),
+        );
+
+        let report = build_final_report(&AgencyConfig::default(), "2026-acme", &files);
+        assert!(report.contains("Performance: 8/10"));
+        assert!(report.contains("Score: 74/100"));
+        assert!(report.contains("Performance: 87/100"));
+        assert!(report.contains("CTA needs work."));
+    }
 }

package/src/security.rs

@@ -1,7 +1,7 @@
 use anyhow::Result;
 use reqwest::header::HeaderMap;
 
-use crate::ui::score_status;
+use crate::ui::{self, score_status, FeedbackTone};
 
 #[derive(Debug, Clone, PartialEq, Eq)]
 pub struct SecurityCheck {
@@ -22,17 +22,15 @@ fn header_present(headers: &HeaderMap, name: &str) -> bool {
 
 fn cookies_have_security_flags(headers: &HeaderMap) -> bool {
     let cookies = headers.get_all("set-cookie");
-    let mut saw_cookie = false;
 
     for cookie in cookies.iter() {
-        saw_cookie = true;
         let value = cookie.to_str().unwrap_or("").to_lowercase();
         if !(value.contains("secure") && value.contains("httponly") && value.contains("samesite")) {
             return false;
         }
     }
 
-    !saw_cookie || true
+    true
 }
 
 pub fn analyze_security_headers(url: &str, headers: &HeaderMap) -> SecurityCheck {
@@ -114,14 +112,90 @@ pub fn check_url(input_url: &str) -> Result<SecurityCheck> {
         .header("accept", "text/html,application/xhtml+xml")
         .send()?;
     Ok(analyze_security_headers(
-        &response.url().to_string(),
+        response.url().as_ref(),
         response.headers(),
     ))
 }
 
 pub fn format_cli(result: &SecurityCheck) -> String {
     let mut output = format!(
-        "\n== Security Check ==\nURL: {}\nScore: {}/100 ({})\n\n== Signals ==\nHTTPS: {}\nHSTS: {}\nCSP: {}\nX-Frame-Options: {}\nReferrer-Policy: {}\nSecure cookies: {}\n\n== Feedback ==\n",
+        "\n{}\n{} {}\n{} {}\n{}\n{}\n{}\n{}\n{}\n{}\n{}\n\n{}\n",
+        ui::frame_line("╭─ Security Check"),
+        ui::frame_line("│ URL  "),
+        result.url,
+        ui::frame_line("│ Score"),
+        ui::score_badge(result.score),
+        ui::frame_line("╰─ Signals"),
+        ui::signal_line(
+            "HTTPS",
+            yes_no(result.https),
+            if result.https {
+                FeedbackTone::Positive
+            } else {
+                FeedbackTone::Critical
+            }
+        ),
+        ui::signal_line(
+            "HSTS",
+            yes_no(result.hsts),
+            if result.hsts {
+                FeedbackTone::Positive
+            } else {
+                FeedbackTone::Warning
+            }
+        ),
+        ui::signal_line(
+            "CSP",
+            yes_no(result.csp),
+            if result.csp {
+                FeedbackTone::Positive
+            } else {
+                FeedbackTone::Critical
+            }
+        ),
+        ui::signal_line(
+            "X-Frame-Options",
+            yes_no(result.frame_options),
+            if result.frame_options {
+                FeedbackTone::Positive
+            } else {
+                FeedbackTone::Warning
+            }
+        ),
+        ui::signal_line(
+            "Referrer-Policy",
+            yes_no(result.referrer_policy),
+            if result.referrer_policy {
+                FeedbackTone::Positive
+            } else {
+                FeedbackTone::Warning
+            }
+        ),
+        ui::signal_line(
+            "Secure cookies",
+            yes_no(result.secure_cookies),
+            if result.secure_cookies {
+                FeedbackTone::Positive
+            } else {
+                FeedbackTone::Critical
+            }
+        ),
+        ui::frame_line("Feedback")
+    );
+
+    for item in &result.feedback {
+        output.push_str(&format!(
+            "{}\n",
+            ui::feedback_line(feedback_tone(item), item)
+        ));
+    }
+
+    output
+}
+
+pub fn format_markdown(result: &SecurityCheck) -> String {
+    let mut output = format!(
+        "# Security Check\n\nURL: {}\nScore: {}/100 ({})\n\n## Signals\n\n- HTTPS: {}\n- HSTS: {}\n- CSP: {}\n- X-Frame-Options: {}\n- Referrer-Policy: {}\n- Secure cookies: {}\n\n## Feedback\n\n",
         result.url,
         result.score,
         score_status(result.score),
@@ -140,10 +214,6 @@ pub fn format_cli(result: &SecurityCheck) -> String {
     output
 }
 
-pub fn format_markdown(result: &SecurityCheck) -> String {
-    format!("# Security Check\n\n{}\n", format_cli(result))
-}
-
 fn yes_no(value: bool) -> &'static str {
     if value {
         "yes"
@@ -152,6 +222,20 @@ fn yes_no(value: bool) -> &'static str {
     }
 }
 
+fn feedback_tone(value: &str) -> FeedbackTone {
+    let value = value.to_lowercase();
+    if value.contains("not using https")
+        || value.contains("content-security-policy")
+        || value.contains("cookies are missing")
+    {
+        FeedbackTone::Critical
+    } else if value.starts_with("missing") {
+        FeedbackTone::Warning
+    } else {
+        FeedbackTone::Positive
+    }
+}
+
 #[cfg(test)]
 mod tests {
     use reqwest::header::{HeaderMap, HeaderValue};

package/src/templates.rs

@@ -36,45 +36,12 @@ fn title_case(value: &str) -> String {
         .join(" ")
 }
 
-fn page_file_name(page: &str) -> String {
-    let raw = if page == "/" {
-        "home".to_string()
-    } else {
-        page.trim_matches('/').to_string()
-    };
-
-    let mut slug = String::new();
-    let mut previous_dash = false;
-
-    for character in raw.chars() {
-        if character.is_ascii_alphanumeric() {
-            slug.push(character.to_ascii_lowercase());
-            previous_dash = false;
-        } else if !previous_dash {
-            slug.push('-');
-            previous_dash = true;
-        }
-    }
-
-    format!("pages/{}.md", slug.trim_matches('-'))
-}
-
 pub fn create_audit_files(audit: &AuditInput) -> BTreeMap<String, String> {
     let mut files = BTreeMap::new();
 
     files.insert("brief.md".to_string(), brief_template(audit));
-    files.insert("status.md".to_string(), status_template());
-    files.insert("scorecard.md".to_string(), scorecard_template());
-    files.insert("checklist.md".to_string(), checklist_template());
+    files.insert("workspace.md".to_string(), workspace_template(audit));
     files.insert("findings.md".to_string(), findings_template());
-    files.insert("report.md".to_string(), report_template(audit));
-    files.insert("video-script.md".to_string(), video_script_template(audit));
-    files.insert("links.md".to_string(), links_template(audit));
-    files.insert("raw-notes.md".to_string(), "# Raw Notes\n\n".to_string());
-
-    for page in &audit.pages {
-        files.insert(page_file_name(page), page_review_template(page));
-    }
 
     files
 }
@@ -95,45 +62,11 @@ pub fn brief_template(audit: &AuditInput) -> String {
     )
 }
 
-pub fn checklist_template() -> String {
-    "# Audit Checklist\n\n## Performance\n\n- [ ] Run PageSpeed on key pages\n- [ ] Check mobile score\n- [ ] Identify LCP element\n- [ ] Check image weight\n- [ ] Check font loading\n- [ ] Check layout shift\n\n## UX and Conversion\n\n- [ ] Above-fold offer is clear\n- [ ] Primary CTA is obvious\n- [ ] User can contact/book within 1 click\n- [ ] Trust signals are visible\n- [ ] Forms are short enough\n- [ ] Navigation supports key journey\n\n## SEO\n\n- [ ] Title tags\n- [ ] Meta descriptions\n- [ ] H1 structure\n- [ ] Internal links\n- [ ] Image alt text\n- [ ] Schema opportunity\n- [ ] Local SEO basics\n".to_string()
-}
-
 pub fn findings_template() -> String {
-    "# Findings\n\n## Finding Template\n\nTitle:\nCategory: Performance / UX / SEO / Conversion / Trust\nSeverity: Critical / High / Medium / Low\nPage:\nEvidence:\nWhy it matters:\nRecommendation:\nEstimated effort:\nBusiness impact:\nScreenshot/video note:\n\n## Critical\n\n### 1. Finding title\n\nEvidence:\n\nWhy it matters:\n\nRecommendation:\n\nEstimated effort:\n\nBusiness impact:\n\n## High\n\n## Medium\n\n## Low\n".to_string()
-}
-
-pub fn scorecard_template() -> String {
-    "# Scorecard\n\nPerformance: /10\nUX clarity: /10\nConversion path: /10\nSEO basics: /10\nTrust signals: /10\nMobile experience: /10\n\nOverall:\n\n## Notes\n\n".to_string()
-}
-
-pub fn status_template() -> String {
-    "# Audit Status\n\n- [ ] Intake complete\n- [ ] Pages reviewed\n- [ ] Performance checked\n- [ ] SEO checked\n- [ ] Findings prioritised\n- [ ] Report drafted\n- [ ] Video recorded\n- [ ] Sent to client\n".to_string()
-}
-
-pub fn page_review_template(page: &str) -> String {
-    format!(
-        "# {} Review\n\nPath: {}\n\n## First Impression\n\n## CTA\n\n## Copy Clarity\n\n## Visual Hierarchy\n\n## Mobile Issues\n\n## SEO Notes\n\n## Performance Notes\n\n## Recommended Fixes\n",
-        title_case(page),
-        page
-    )
-}
-
-pub fn report_template(audit: &AuditInput) -> String {
-    format!(
-        "# {} Website Audit\n\nWebsite: {}\nGoal: {}\n\n## Executive Summary\n\n## Priority Fixes\n\n1.\n2.\n3.\n\n## Performance\n\n## UX and Conversion\n\n## SEO\n\n## Recommended Next Step\n",
-        audit.client_name, audit.url, audit.goal
-    )
-}
-
-pub fn video_script_template(audit: &AuditInput) -> String {
-    format!(
-        "# {} Video Script\n\n## Opening\n\nWhat I reviewed and what matters most.\n\n## Biggest Conversion Issue\n\n## Performance Issue\n\n## SEO or Structure Issue\n\n## Close\n\nSummarise top 3 fixes and recommended next step.\n",
-        audit.client_name
-    )
+    "# Findings\n\n## Critical\n\n## High\n\n## Medium\n\n## Low\n\n## Finding Template\n\nTitle:\nCategory: Performance / UX / SEO / Conversion / Trust\nSeverity: Critical / High / Medium / Low\nPage:\nEvidence:\nWhy it matters:\nRecommendation:\nEstimated effort:\nBusiness impact:\nScreenshot/video note:\n".to_string()
 }
 
-pub fn links_template(audit: &AuditInput) -> String {
+pub fn workspace_template(audit: &AuditInput) -> String {
     let encoded = urlencoding(&audit.url);
     let pages = audit
         .pages
@@ -147,13 +80,27 @@ pub fn links_template(audit: &AuditInput) -> String {
         .collect::<Vec<_>>()
         .join("\n");
 
+    let page_reviews = audit
+        .pages
+        .iter()
+        .map(|page| {
+            format!(
+                "### {} Review\n\nPath: {}\n\nFirst impression:\n\nCTA:\n\nCopy clarity:\n\nVisual hierarchy:\n\nMobile issues:\n\nSEO notes:\n\nPerformance notes:\n\nRecommended fixes:\n",
+                title_case(page),
+                page
+            )
+        })
+        .collect::<Vec<_>>()
+        .join("\n");
+
     format!(
-        "# Audit Links\n\n## Test Links\n\n- PageSpeed: https://pagespeed.web.dev/analysis?url={}\n- Rich Results: https://search.google.com/test/rich-results?url={}\n- Meta Preview: https://metatags.io/?url={}\n\n## Pages\n\n{}\n\n## Competitors\n\n{}\n",
+        "# Audit Workspace\n\n## Status\n\n- [ ] Intake complete\n- [ ] Pages reviewed\n- [ ] Performance checked\n- [ ] SEO checked\n- [ ] Findings prioritised\n- [ ] Report drafted\n- [ ] Video recorded\n- [ ] Sent to client\n\n## Scorecard\n\nPerformance: /10\nUX clarity: /10\nConversion path: /10\nSEO basics: /10\nTrust signals: /10\nMobile experience: /10\n\nOverall:\n\nNotes:\n\n## Review Checklist\n\n### Performance\n\n- [ ] Check mobile score\n- [ ] Identify LCP element\n- [ ] Check image weight\n- [ ] Check font loading\n- [ ] Check layout shift\n\n### UX and Conversion\n\n- [ ] Above-fold offer is clear\n- [ ] Primary CTA is obvious\n- [ ] User can contact/book within 1 click\n- [ ] Trust signals are visible\n- [ ] Forms are short enough\n- [ ] Navigation supports key journey\n\n### SEO\n\n- [ ] Title tags\n- [ ] Meta descriptions\n- [ ] H1 structure\n- [ ] Internal links\n- [ ] Image alt text\n- [ ] Schema opportunity\n- [ ] Local SEO basics\n\n## Test Links\n\n- PageSpeed: https://pagespeed.web.dev/analysis?url={}\n- Rich Results: https://search.google.com/test/rich-results?url={}\n- Meta Preview: https://metatags.io/?url={}\n\n## Pages To Review\n\n{}\n\n## Competitors\n\n{}\n\n## Automated Check\n\nNot run yet.\n\n## Security Check\n\nNot run yet.\n\n## Lighthouse Check\n\nNot run yet.\n\n## Page Reviews\n\n{}\n\n## Raw Notes\n\n",
         encoded,
         encoded,
         encoded,
         if pages.is_empty() { "- None provided".to_string() } else { pages },
-        markdown_list(&audit.competitors)
+        markdown_list(&audit.competitors),
+        if page_reviews.is_empty() { "No page reviews provided yet.".to_string() } else { page_reviews }
     )
 }
 
@@ -189,9 +136,11 @@ mod tests {
     fn create_audit_files_contains_expected_workspace() {
         let files = create_audit_files(&audit());
         assert!(files.contains_key("brief.md"));
-        assert!(files.contains_key("scorecard.md"));
-        assert!(files.contains_key("pages/home.md"));
-        assert!(files.contains_key("pages/pricing.md"));
+        assert!(files.contains_key("workspace.md"));
+        assert!(files.contains_key("findings.md"));
+        assert_eq!(files.len(), 3);
         assert!(files["brief.md"].contains("Business type: Dental clinic"));
+        assert!(files["workspace.md"].contains("### Homepage Review"));
+        assert!(files["workspace.md"].contains("## Automated Check"));
     }
 }