feat: share todo with email

[jeffstedt]

Added POST /api/todos/:id/share endpoint that will allow one to share a todo with one or many people over email.

Edit: Secretly testing Copilot 👀

[Copilot]

Copilot reviewed 6 out of 7 changed files in this pull request and generated 4 comments.

Files not reviewed (1)

• .env: Language not supported

[Copilot]

The test URL for sharing a todo is missing the required todo id parameter as per the endpoint '/api/todos/:id/share'. Please update the URL to include an id, for example, '/api/todos/1/share'.

Suggested change

	Deno.test("POST /api/todos/share shares a todo with a user", async () => {
	const shareData = { todoId: 1, userId: 2 };
	const response = await mockRequest("POST", "http://localhost:8080/api/todos/share", shareData);
	Deno.test("POST /api/todos/:id/share shares a todo with a user", async () => {
	const shareData = { todoId: 1, userId: 2 };
	const response = await mockRequest("POST", "http://localhost:8080/api/todos/1/share", shareData);

[Copilot]

The deduplication check for shared recipients mentioned in the comment is not implemented. Consider adding logic to prevent sharing the same todo with duplicate email addresses.

Suggested change

	// Important: Check that todo hasn't been shared with the same user
	// Important: Check that todo hasn't been shared with the same user
	const deduplicatedSharedWith = Array.from(new Set(validatedBody.sharedWith));

[Copilot]

The share endpoint path appears to be misspelled as '/shar'. Please correct it to '/share' to match the implementation.

Suggested change

	- POST /api/todos/:id/shar - Shares an existing todo
	- POST /api/todos/:id/share - Shares an existing todo

[Copilot]

A hardcoded default email value is used when sharedWith is not provided. Consider removing or parameterizing this default to avoid unexpected behavior in production.

Suggested change

	updatedAt: new Date().toISOString(),
	sharedWith: inputTodo.sharedWith ?? ["[email protected]"],
	};
	updatedAt: new Date().toISOString()
	};
	if (inputTodo.sharedWith) {
	this.todos[inputId].sharedWith = inputTodo.sharedWith;
	}