refactor: remove list callback implementation for entry resource template

Author: kentcdodds

exercises/03.resources/03.problem.list/README.mdx

@@ -1,7 +1,7 @@
 # Resource Templates List
 
-👨‍💼 Our users want to be able to see a list of all entries and tags. So let's
-implement the `list` callback for our resource templates.
+👨‍💼 Our users want to be able to see a list of all tags. So let's implement the
+`list` callback for our resource templates.
 
 ```ts
 import { ResourceTemplate } from '@modelcontextprotocol/sdk/server/mcp.js'
@@ -27,8 +27,8 @@ agent.server.registerResource(
 ```
 
 Notice how the `list` callback queries the database and returns a resource
-listing for each name. This pattern is exactly what you'll use to expose entries
-and tags from your own database.
+listing for each name. This pattern is exactly what you'll use to expose tags
+from your own database.
 
 <callout-success>
 	The reason for the `list` callback is to make it easy for clients to get a
@@ -37,11 +37,4 @@ and tags from your own database.
 	and type in the `list` callback).
 </callout-success>
 
-Your goal in this step:
-
-- Use resource templates to expose entries and tags from your database, each
-  accessible by a unique URI.
-- Make sure clients can list all available entries and tags (using the `list`
-  callback), and read the details for any specific one.
-
 To test this out, click the "List Resources" button in the MCP Inspector.

exercises/03.resources/03.problem.list/src/resources.ts

@@ -53,8 +53,8 @@ export async function initializeResources(agent: EpicMeMCP) {
 	agent.server.registerResource(
 		'entry',
 		new ResourceTemplate('epicme://entries/{id}', {
-			// 🐨 implement this list callback to get all entries (💰 agent.db.getEntries())
-			// 🐨 return an object with a "resources" property that is an array of resource listings (💰 each object has a name, uri, and mimeType).
+			// 🦉 there are probably too many journal entries to list them all, so
+			// we'll not implement the list callback for this resource template.
 			list: undefined,
 		}),
 		{

exercises/03.resources/03.solution.list/src/resources.ts

@@ -60,16 +60,7 @@ export async function initializeResources(agent: EpicMeMCP) {
 	agent.server.registerResource(
 		'entry',
 		new ResourceTemplate('epicme://entries/{id}', {
-			list: async () => {
-				const entries = await agent.db.getEntries()
-				return {
-					resources: entries.map((entry) => ({
-						name: entry.title,
-						uri: `epicme://entries/${entry.id}`,
-						mimeType: 'application/json',
-					})),
-				}
-			},
+			list: undefined,
 		}),
 		{
 			title: 'Journal Entry',

exercises/03.resources/04.problem.completion/src/resources.ts

@@ -63,19 +63,10 @@ export async function initializeResources(agent: EpicMeMCP) {
 	agent.server.registerResource(
 		'entry',
 		new ResourceTemplate('epicme://entries/{id}', {
+			list: undefined,
 			// 🐨 add a `complete` callback for the `id` parameter
 			// 🐨 accept a value and return an array of strings of ids that include the value
 			// 💰 const entries = await agent.db.getEntries()
-			list: async () => {
-				const entries = await agent.db.getEntries()
-				return {
-					resources: entries.map((entry) => ({
-						name: entry.title,
-						uri: `epicme://entries/${entry.id}`,
-						mimeType: 'application/json',
-					})),
-				}
-			},
 		}),
 		{
 			title: 'Journal Entry',

exercises/03.resources/04.solution.completion/src/resources.ts

@@ -68,6 +68,7 @@ export async function initializeResources(agent: EpicMeMCP) {
 	agent.server.registerResource(
 		'entry',
 		new ResourceTemplate('epicme://entries/{id}', {
+			list: undefined,
 			complete: {
 				async id(value) {
 					const entries = await agent.db.getEntries()
@@ -76,16 +77,6 @@ export async function initializeResources(agent: EpicMeMCP) {
 						.filter((id) => id.includes(value))
 				},
 			},
-			list: async () => {
-				const entries = await agent.db.getEntries()
-				return {
-					resources: entries.map((entry) => ({
-						name: entry.title,
-						uri: `epicme://entries/${entry.id}`,
-						mimeType: 'application/json',
-					})),
-				}
-			},
 		}),
 		{
 			title: 'Journal Entry',

exercises/04.resource-tools/01.problem.embedded/src/resources.ts

@@ -68,6 +68,7 @@ export async function initializeResources(agent: EpicMeMCP) {
 	agent.server.registerResource(
 		'entry',
 		new ResourceTemplate('epicme://entries/{id}', {
+			list: undefined,
 			complete: {
 				async id(value) {
 					const entries = await agent.db.getEntries()
@@ -76,16 +77,6 @@ export async function initializeResources(agent: EpicMeMCP) {
 						.filter((id) => id.includes(value))
 				},
 			},
-			list: async () => {
-				const entries = await agent.db.getEntries()
-				return {
-					resources: entries.map((entry) => ({
-						name: entry.title,
-						uri: `epicme://entries/${entry.id}`,
-						mimeType: 'application/json',
-					})),
-				}
-			},
 		}),
 		{
 			title: 'Journal Entry',

exercises/04.resource-tools/01.solution.embedded/src/resources.ts

@@ -68,6 +68,7 @@ export async function initializeResources(agent: EpicMeMCP) {
 	agent.server.registerResource(
 		'entry',
 		new ResourceTemplate('epicme://entries/{id}', {
+			list: undefined,
 			complete: {
 				async id(value) {
 					const entries = await agent.db.getEntries()
@@ -76,16 +77,6 @@ export async function initializeResources(agent: EpicMeMCP) {
 						.filter((id) => id.includes(value))
 				},
 			},
-			list: async () => {
-				const entries = await agent.db.getEntries()
-				return {
-					resources: entries.map((entry) => ({
-						name: entry.title,
-						uri: `epicme://entries/${entry.id}`,
-						mimeType: 'application/json',
-					})),
-				}
-			},
 		}),
 		{
 			title: 'Journal Entry',

exercises/04.resource-tools/02.problem.linked/src/resources.ts

@@ -68,6 +68,7 @@ export async function initializeResources(agent: EpicMeMCP) {
 	agent.server.registerResource(
 		'entry',
 		new ResourceTemplate('epicme://entries/{id}', {
+			list: undefined,
 			complete: {
 				async id(value) {
 					const entries = await agent.db.getEntries()
@@ -76,16 +77,6 @@ export async function initializeResources(agent: EpicMeMCP) {
 						.filter((id) => id.includes(value))
 				},
 			},
-			list: async () => {
-				const entries = await agent.db.getEntries()
-				return {
-					resources: entries.map((entry) => ({
-						name: entry.title,
-						uri: `epicme://entries/${entry.id}`,
-						mimeType: 'application/json',
-					})),
-				}
-			},
 		}),
 		{
 			title: 'Journal Entry',

exercises/04.resource-tools/02.solution.linked/src/resources.ts

@@ -68,6 +68,7 @@ export async function initializeResources(agent: EpicMeMCP) {
 	agent.server.registerResource(
 		'entry',
 		new ResourceTemplate('epicme://entries/{id}', {
+			list: undefined,
 			complete: {
 				async id(value) {
 					const entries = await agent.db.getEntries()
@@ -76,16 +77,6 @@ export async function initializeResources(agent: EpicMeMCP) {
 						.filter((id) => id.includes(value))
 				},
 			},
-			list: async () => {
-				const entries = await agent.db.getEntries()
-				return {
-					resources: entries.map((entry) => ({
-						name: entry.title,
-						uri: `epicme://entries/${entry.id}`,
-						mimeType: 'application/json',
-					})),
-				}
-			},
 		}),
 		{
 			title: 'Journal Entry',

exercises/05.prompts/01.problem.prompts/src/resources.ts

@@ -68,6 +68,7 @@ export async function initializeResources(agent: EpicMeMCP) {
 	agent.server.registerResource(
 		'entry',
 		new ResourceTemplate('epicme://entries/{id}', {
+			list: undefined,
 			complete: {
 				async id(value) {
 					const entries = await agent.db.getEntries()
@@ -76,16 +77,6 @@ export async function initializeResources(agent: EpicMeMCP) {
 						.filter((id) => id.includes(value))
 				},
 			},
-			list: async () => {
-				const entries = await agent.db.getEntries()
-				return {
-					resources: entries.map((entry) => ({
-						name: entry.title,
-						uri: `epicme://entries/${entry.id}`,
-						mimeType: 'application/json',
-					})),
-				}
-			},
 		}),
 		{
 			title: 'Journal Entry',