fix: remove story-bank.md from git tracking

[mskadu]

story-bank.md is user-generated content that accumulates personal interview stories. It was tracked in the initial STAR+R framework commit but should be user-managed, like other user data files (cv.md, config/profile.yml, etc.).

This was originally part of PR #707 but split out per review feedback.

Summary by CodeRabbit

• Documentation
  • Removed Story Bank template content and instructions.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro

Run ID: 16a289b7-a83e-4ae5-84c5-a00c08fc07d3

📥 Commits

Reviewing files that changed from the base of the PR and between a7f8702 and 01d4b5c.

📒 Files selected for processing (1)

• interview-prep/story-bank.md

💤 Files with no reviewable changes (1)

• interview-prep/story-bank.md

---

📝 Walkthrough

Walkthrough

The PR removes 26 lines of content from interview-prep/story-bank.md, including the "How it works" instructional section and template comments that guided users on appending new STAR+R stories to the document.

Changes

Cohort / File(s)	Summary
Story bank content removal interview-prep/story-bank.md	Removed "How it works" section and template/placeholder comments instructing story bank usage; 26 lines deleted.

🎯 1 (Trivial) | ⏱️ ~2 minutes

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Title check	⚠️ Warning	The title states 'remove story-bank.md from git tracking' but the actual change only removes content from the file; git tracking is not addressed in the changeset.	Update the title to 'remove story-bank.md content' or similar to accurately reflect that the file's STAR+R content is being deleted, not git tracking being removed.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
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

---

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.}

[santifer]

Clean split, exactly as discussed — and with #707's gitignore rules in place this closes the loop: user stories stay on the user's machine, period. Merged! 🚀