8fc0974 · hyper-wallet

hyper-wallet commits / 8fc0974

Fix routing: Use startsWith for all routes to support POST with path params

CRITICAL FIX: POST /api/contacts/:id/addresses was failing with "route not found"

Before:
- POST routes used exact match only
- req.url=/api/contacts/123/addresses didn't match routePath=/api/contacts/

After:
- All routes (GET and POST) use startsWith
- Parse path params by splitting remaining path
- Pass params as spread arguments to handler

This fixes:
- Adding Ethereum address to contact (POST /api/contacts/:id/addresses)
- Removing address from contact (DELETE /api/contacts/:id/addresses/:chain)

User report: "if i paste storyteller's eth address into the first contact it still fails with route not found"

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>

8fc0974d31b00e5a164ce9478418c310f39c28c6

by plankatron <fleeky@gmail.com> · 2026-01-27T22:48:18.000Z

Changed files (1) +5 -6

wallet-server.cjs +5 -6

…	…	@@ -1941,15 +1941,14 @@ async function startServer() {
1941	1941	for (const [routeKey, handler] of Object.entries(routes)) {
1942	1942	const [method, routePath] = routeKey.split(' ')
1943	1943
1944		// For POST routes, match exactly; for GET routes with params, use startsWith
1945		const isMatch = (method === 'POST')
1946		? (req.method === method && req.url === routePath)
1947		: (req.method === method && req.url.startsWith(routePath))
	1944	// Use startsWith for all routes to support path parameters
	1945	const isMatch = req.method === method && req.url.startsWith(routePath)
1948	1946
1949	1947	if (isMatch) {
1950		// Extract parameter (e.g., chain from /api/balance/bitcoin)
	1948	// Extract parameters (e.g., :id/addresses from /api/contacts/:id/addresses)
1951	1949	const param = req.url.substring(routePath.length)
1952		await handler(req, res, param)
	1950	const params = param ? param.split('/').filter(p => p) : []
	1951	await handler(req, res, ...params)
1953	1952	handled = true
1954	1953	break
1955	1954	}