feat(save): save markdown and websites from UI for chat creations

[ArslanSaleem]

Important

This PR adds functionality to save markdown files from the UI to the backend, including new backend routes, services, and frontend components for artifact management.

• Backend:
  • Adds artifacts.py route to handle artifact saving in routes.
  • Implements ArtifactsService in services/artifacts.py to process markdown files and extract linked files.
  • Introduces FilesService in services/files.py for file path validation and retrieval.
  • Updates main.py to include artifacts router.
• Frontend:
  • Adds SaveArtifactButton component in save-artifact-button.tsx for saving artifacts from the UI.
  • Integrates @radix-ui/react-dialog for dialog management in dialog.tsx.
  • Updates content-sidebar.tsx to include save button for markdown files.
  • Adds saveArtifact function in api/artifacts.ts to handle API requests for saving artifacts.
  • Updates package.json to include @radix-ui/react-dialog dependency.

^{This description was created by for fd240e8. You can customize this summary. It will automatically update as commits are pushed.}

[ellipsis-dev]

Caution

Changes requested ❌

Reviewed everything up to fd240e8 in 2 minutes and 14 seconds. Click for details.

• Reviewed 753 lines of code in 11 files
• Skipped 1 files when reviewing.
• Skipped posting 1 draft comments. View those below.
• Modify your settings and rules to customize what types of comments Ellipsis leaves. And don't forget to react with 👍 or 👎 to teach Ellipsis.

1. examples/ui/frontend/src/components/content-sidebar.tsx:775

• Draft comment:
  The string manipulation for determining the download file extension (replacing 'md' with 'pdf') is brittle; consider refactoring for clarity and robustness.
• Reason this comment was not posted:
  Comment was not on a location in the diff, so it can't be submitted as a review comment.

Workflow ID: wflow_npzhKHSJGLggqlc7

^{You can customize by changing your verbosity settings, reacting with 👍 or 👎, replying to comments, or adding code review rules.}

[ellipsis-dev]

Unused import: HTTPBearer is imported but never used; consider removing it.

Suggested change

from fastapi.security import HTTPBearer

[ellipsis-dev]

Avoid using 'type' as a parameter name to prevent shadowing the built-in; consider renaming it (e.g. artifactType).

^{This comment was generated because it violated a code review rule: mrule_jwFVvydpsvYdje26.}

[ellipsis-dev]

The return type of get_file_from_env is declared as 'str' but it returns a tuple (content_bytes, mime_type); update the type annotation accordingly.

Suggested change

	async def get_file_from_env(file_path: str, env: BaseEnv) -> str:
	async def get_file_from_env(file_path: str, env: BaseEnv) -> tuple[bytes, str]:

[ellipsis-dev]

Custom exception 'FileNotFoundError' conflicts with the built-in exception; consider renaming it (e.g. CustomFileNotFoundError) to avoid confusion.

^{This comment was generated because it violated a code review rule: mrule_jwFVvydpsvYdje26.}

[ellipsis-dev]

Ensure new API routes and service functions (e.g. artifacts endpoints) are accompanied by proper unit tests.

^{This comment was generated because it violated a code review rule: mrule_q7uemN6whkfXoUH0.}

[ellipsis-dev]

The dialog title 'Save Completion' may be misleading; consider renaming it to 'Save Artifact' to match the feature purpose.

^{This comment was generated because it violated a code review rule: mrule_kO2c9rTcwkbqZtg4.}