Resolve editing mistake and update some backend code

[GeekTrainer]

This pull request introduces improvements to end-to-end (E2E) testing, developer documentation, and configuration for the crowdfunding platform. The most significant changes include enhanced Playwright E2E tests for reliability and maintainability, updates to documentation for better guidance on using Copilot and scripts, and configuration tweaks to optimize local and CI development workflows.

E2E Test Improvements:

• Refactored Playwright E2E tests in client/e2e-tests/games.spec.ts and client/e2e-tests/home.spec.ts to use more semantic and robust locators (getByTestId, getByRole, getByText), improving reliability and readability. Tests now check for non-empty content and handle navigation more gracefully. [1] [2] [3] [4]
• Updated Playwright configuration in client/playwright.config.ts to use CI-appropriate settings (single worker, GitHub reporter), and added screenshot/video capture on failure, as well as standardized timeouts for actions and navigation. Server reuse is now always enabled for faster local runs. [1] [2]

Documentation and Copilot Instructions:

• Added an "Agent notes" section to .github/copilot-instructions.md with specific guidance for Copilot agent usage, and clarified script usage instructions. [1] [2]
• Expanded docs/3-custom-instructions.md with step-by-step guidance for using Copilot custom instructions and agent mode, including new notes, screenshots, and resource links. [1] [2]

Developer Experience and Configuration:

• Updated .vscode/settings.json to set a higher request limit for Copilot agent, supporting more interactive sessions.
• Ensured Playwright is installed automatically during environment setup by adding npx playwright install to scripts/setup-env.sh.
• Improved TypeScript configuration in client/tsconfig.json by excluding node_modules for better performance and accuracy.

Frontend and Dependency Updates:

• Changed Tailspin Toys branding in client/src/components/Header.astro from an h1 to a div for better semantic structure and updated related tests. [1] [2]
• Upgraded dependencies in client/package.json, including Astro and TypeScript, and added @astrojs/check for improved type safety.

Minor Fixes:

• Removed an unused variable in client/src/pages/game/[id].astro. (client/src/pages/game/[id].astroL9)

[Copilot]

Pull Request Overview

This pull request introduces comprehensive improvements to the crowdfunding platform's testing infrastructure, developer documentation, and configuration settings. The changes focus on enhancing the reliability of end-to-end tests, improving developer experience with better Copilot integration, and optimizing the development workflow.

• Refactored Playwright E2E tests to use more semantic and robust locators for improved reliability
• Enhanced developer documentation with detailed guidance for using Copilot custom instructions and agent mode
• Updated development environment configuration to optimize local and CI workflows

Reviewed Changes

Copilot reviewed 11 out of 12 changed files in this pull request and generated 1 comment.

Show a summary per file

File	Description
scripts/setup-env.sh	Added automatic Playwright installation to environment setup
docs/3-custom-instructions.md	Expanded documentation with complete step-by-step guidance for Copilot usage
client/tsconfig.json	Excluded node_modules from TypeScript compilation for better performance
client/src/pages/game/[id].astro	Removed unused variable to clean up code
client/src/components/Header.astro	Changed site branding from h1 to div for better semantic structure
client/playwright.config.ts	Enhanced configuration with CI-appropriate settings and failure capture
client/package.json	Updated dependencies including Astro and TypeScript versions
client/e2e-tests/home.spec.ts	Refactored tests to use semantic locators and improved test structure
client/e2e-tests/games.spec.ts	Updated tests with more robust locators and better error handling
.vscode/settings.json	Increased Copilot agent request limit for better development experience
.github/copilot-instructions.md	Added agent-specific notes and clarified script usage guidance

Files not reviewed (1)

• client/package-lock.json: Language not supported

[Copilot]

The logic should check that both publisher AND category exist, not OR. The comment above states 'Check that either publisher or category (or both) are present' but the test expects both to be present based on the original logic.

See below for a potential fix:

    // Check that both publisher and category are present
    const publisherExists = await page.getByTestId('game-details-publisher').isVisible();
    const categoryExists = await page.getByTestId('game-details-category').isVisible();
    expect(publisherExists && categoryExists).toBeTruthy();

[peckjon]

LGTM!