Staging rebase

[Hackshaven]

This pull request updates the README.md file to improve clarity, structure, and user guidance while also adding a security check in the src/digital_persona/api.py file to validate file paths. Below is a summary of the most important changes:

Documentation Improvements:

• The README.md file has been rewritten for better readability and organization. It now includes clearer descriptions of the project, its mission, features, architecture, and development setup. New sections such as "What is a Digital Persona?" and "Features and Architecture" have been added to provide a more comprehensive overview.
• A detailed development setup guide has been added, explaining how to configure OpenAI and Ollama models, run the development CLI, and use the devcontainer for local development.
• The "License" and "Documentation" sections have been streamlined, with links to GitHub Pages for research and schema documentation.

Security Enhancements:

• A new security check has been added to the complete_interview function in src/digital_persona/api.py to ensure that output file paths are restricted to the OUTPUT_DIR directory. This prevents potential path traversal vulnerabilities.

[Copilot]

Pull Request Overview

This PR enhances security by validating output file paths in the interview API and overhauls the README for improved structure, clarity, and user guidance.

• Added a .resolve() call and path traversal check in complete_interview to restrict output files to OUTPUT_DIR
• Fully rewrote and reorganized README.md with new sections (mission, features, setup) and streamlined content

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File	Description
src/digital_persona/api.py	Added absolute resolution of out_path and a security check to prevent path traversal
README.md	Reorganized documentation, added new sections, updated examples

[Copilot]

Instead of a string prefix check, resolve OUTPUT_DIR and use out_path.is_relative_to(resolved_output_dir) to robustly verify that out_path is inside OUTPUT_DIR, preventing bypasses via symlinks or similar prefixes.

[Copilot]

[nitpick] The triple-quoted string preserves line breaks, so the explicit "\n" is redundant; consider removing it to simplify the example.

Suggested change

	data = """Email: I'm looking forward to the team retreat next month.\n
	data = """Email: I'm looking forward to the team retreat next month.