fix: resolve all 12 bugs in launchdesk debugging lab

[Pamindu Virunjith (pamindu-virunjith)]

Fix all 12 bugs in the LaunchDesk debugging lab
Resolved all intentional bugs across form handling, filtering, scoring, storage, and export.

Fixes,

1. handleAddChek typo broke the add form submit handler.
2. && logic allowed submission with only one required field filled.
3. Search only matched against owner — now checks all fields.
4. Status filter compared priority instead of status.
5. In Progress badge unstyled due to space in the generated CSS class name.
6. Readiness score filtered for "Complete" instead of "Fixed".
7. Due This Week counted far-future items — condition was inverted.
8. Delete button silent — JS looked for data-delete-id, HTML used data-remove-id.
9. Status changes only re-rendered the row — score, filters, and storage not updated.
10. Save and load used different localStorage keys, discarding all changes on refresh.
11. Demo reset fetched launch-seed.json — file is launch-checks.json.
12. CSV export referenced check.name instead of check.title, leaving Title column blank.

Files changed,

• js/app.js - all fixes
• No logic changes to HTML or CSS

Summary by CodeRabbit

• Bug Fixes
  • Resolved checklist data persistence with unified storage management
  • Corrected form validation and submission handling for new items
  • Enhanced search to filter across multiple fields
  • Fixed status display styling and metrics calculations
  • Improved delete functionality and data consistency
  • Status changes now persist with immediate UI refresh
  • Corrected CSV export and demo data loading paths

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 677b3dea-0853-4c38-8a0a-9ed16b9d8430

📥 Commits

Reviewing files that changed from the base of the PR and between ccb0d19 and 6debec3.

📒 Files selected for processing (1)

• js/app.js

---

📝 Walkthrough

Walkthrough

The PR fixes multiple bugs in the launch-checklist application: consolidating storage operations to use a single key, correcting form submission and validation, reworking filter search across multiple fields, fixing metrics calculations, correcting row deletion and status updates, and repairing CSV export and demo data loading.

Changes

Launch Checklist Bug Fixes

Layer / File(s)	Summary
Data Persistence Foundation js/app.js	Storage operations now use a unified STORAGE_KEY constant for both loading and saving checklist state.
Form Input and Validation js/app.js	Form submit handler wired to correct handleAddCheck function; add-check validation now blocks submission when either title or category is missing.
Filtering and Status Display js/app.js	Search now matches across owner, title, category, priority, and status fields; status filter applied to selectedStatus; status badge CSS classes generated by slugifying spaces in status names.
Metrics Calculation and Status Persistence js/app.js	Metrics corrected to count "Fixed" as completed status, compute "critical open" from non-fixed items, and count "due soon" within 0–7 day window; status changes now persist to storage, re-apply filters, and log activity.
Row Deletion and CSV Export js/app.js	Delete handling corrected to read data-remove-id attribute; CSV export corrected to write check.title for Title column; demo reset corrected to fetch data/launch-checks.json.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

Poem

A checklist once broken, now mended with care,
Storage keys unified, persistence fair.
Search through the fields, metrics align,
Delete buttons fixed, export CSV fine! 🐰✨

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The PR title directly and accurately summarizes the main change: fixing all 12 intentional bugs in the LaunchDesk debugging lab exercise, which aligns with the AI-generated summary and PR objectives.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests

Warning

Review ran into problems

🔥 Problems

Git: Failed to clone repository. Please run the @coderabbitai full review command to re-trigger a full review. If the issue persists, set path_filters to include or exclude specific files.

Tip

💬 Introducing Slack Agent: The best way for teams to turn conversations into code.

Slack Agent is built on CodeRabbit's deep understanding of your code, so your team can collaborate across the entire SDLC without losing context.

• Generate code and open pull requests
• Plan features and break down work
• Investigate incidents and troubleshoot customer tickets together
• Automate recurring tasks and respond to alerts with triggers
• Summarize progress and report instantly

Built for teams:

• Shared memory across your entire org—no repeating context
• Per-thread sandboxes to safely plan and execute work
• Governance built-in—scoped access, auditability, and budget controls

One agent for your entire SDLC. Right inside Slack.

👉 Get started

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}