Open PR: simplify-client-app → main

[Copilot]

PR #179 from simplify-client-app into main was already open but lacked a comprehensive description. This agent session confirmed the branch is complete and covers all intended changes.

Changes on simplify-client-app

• Project restructure — client/, server/, scripts/ moved under app/; Dependabot config, .gitignore, and workshop content updated accordingly
• Client simplification — Svelte removed; components converted to pure Astro with SSR data fetching; dead deps (autoprefixer, postcss, flask-cors) dropped; Header reduced to zero client JS
• Pagination — page/per_page query params added to GET /api/dogs and wired into the homepage
• E2e tests — Mock API replaced with real Flask server backed by a seeded test database
• Dev tooling — .dev/ directory with Dockerfile and startup script for Copilot sandbox environments; startup script genericized via external sandbox.json config
• Misc — Google Fonts removed (system font stack), Astro telemetry disabled, Vite server.fs.allow fix for symlinked node_modules, workshop content and line-number references updated

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

• https://api.github.com/graphql
  • Triggering command: /usr/bin/gh gh pr list --repo github-samples/pets-workshop (http block)
  • Triggering command: `/usr/bin/gh gh pr create --repo github-samples/pets-workshop --base main --head simplify-client-app --title Simplify client app: restructure into app/ directory, pure Astro with SSR, pagination, improved e2e tests, and dev tooling --body ## Summary

This PR consolidates several improvements to the pets-workshop sample project:

1. Restructure project into app/ directory

• Moved client/, server/, and scripts/ directories under a centralized app/ folder
• Updated all reference` (http block)

• Triggering command: /usr/bin/gh gh auth status (http block)
• https://api.github.com/repos/github-samples/pets-workshop/pulls
  • Triggering command: /usr/bin/curl curl -s -X POST -H Authorization: token ****** -H Accept: application/vnd.github.v3+json -H Content-Type: application/json REDACTED -d { "title": "Simplify client app: restructure into app/ directory, pure Astro SSR, pagination, e2e improvements, and dev tooling", "body": "## Summary\n\nThis PR consolidates several improvements to the pets-workshop sample project:\n\n### 1. Restr (http block)
• https://api.github.com/user
  • Triggering command: /usr/bin/curl curl -s -o /dev/null -w %{http_code} -H Authorization: token ****** REDACTED (http block)

If you need me to access, download, or install something from one of these locations, you can either:

• Configure Actions setup steps to set up my environment, which run before the firewall is enabled
• Add the appropriate URLs or hosts to the custom allowlist in this repository's Copilot coding agent settings (admins only)

Original prompt

Open a pull request from the existing simplify-client-app branch into main.

This branch contains the following changes that have already been committed:

1. Restructure project into app/ directory — Moved client/, server/, and scripts/ directories under a centralized app/ folder. Updated all references in dependabot config, gitignore, workshop content, and filepath comments.

2. Remove old top-level client/, server/, scripts/ directories — Cleanup after the migration to app/.

3. Simplify client — Removed Svelte, converted to pure Astro with SSR data fetching. Removed dead dependencies (autoprefixer, postcss, flask-cors), unused starter assets, and simplified the Header component to zero client JS.

4. Add pagination to dogs API and homepage with page/per_page query parameters.

5. Improve e2e tests — Replaced mock API with real Flask server using a seeded test database.

6. Add dev tooling for Copilot sandbox environment (.dev/ directory with Dockerfile and startup script).

7. Genericize sandbox script with external config (sandbox.json) so the startup script is reusable across projects.

8. Other improvements: Removed Google Fonts (use system font stack), disabled Astro telemetry, fixed Vite serving errors for symlinked node_modules, updated workshop content to reflect pure Astro app, fixed content alignment and line number references.

Do NOT make any code changes. Simply open a PR from the simplify-client-app branch to main with an appropriate title and description summarizing the above changes.

The following is the prior conversation context from the user's chat exploration (may be truncated):

User: I had created a branch (or i think i did) to migrate the sample project to a centralized app folder. does that branch still exist?

Assistant: I don't see a branch with an obvious name like "centralized-app-folder" or "migrate-app," but there are a couple of branches that could match what you're describing based on their names. The most likely candidates are:

Branch	Description
restructure-github-actions-workshop	The name suggests restructuring/reorganizing the project — could involve centralizing into an app folder.
simplify-client-app	Could involve consolidating/migrating the client app into a centralized location.
modernization	A general modernization effort that might include folder restructuring.

Here's the full list of non-Dependabot branches in the repo for reference:

• copilot/update-dependencies-from-dependabot
• main (protected)
• modernization
• msbuild-25
• msignite-25
• restructure-github-actions-workshop
• revert-20-peckjon-patch-7
• simplify-client-app
• update-workshop

Do any of those ring a bell? If you'd like, I can look at the commits or file changes on any of these branches to help you identify which one has the app folder migration work.

User: the simplify-client-app one. can you look in there?

Assistant: Yes! The simplify-client-app branch is definitely your migration branch. Here's what I found:

✅ This is the one

The commit history tells the story clearly. The key commits on this branch (all by you, with Copilot co-authoring) are:

Date	Commit	What it does
Mar 1	5e2adb4	Simplify client: remove Svelte, use pure Astro with SSR
Mar 2	6f610c0	Restructure project into app/ directory — moved client/, server/, and scripts/ under app/
Mar 2	d84b9ae	Remove old client/, server/, scripts/ directories — cleanup after the move
Mar 2	4d9a59c	Add dev tooling for Copilot sandbox (.dev/ directory)
Mar 3	042df81	Genericize sandbox script with external config (sandbox.json)

Along the way it also added pagination, e2e tests with a real Flask server, removed Google Fonts, and updated workshop content.

Current state vs main

On main, the top-level structure still has separate client/, server/, scripts/ directories. On simplify-client-app, those have been consolidated under app/ with a .dev/ directory for tooling — exactly the centralized app folder migration you described.

The branch's last commit was March 3, so it's about 6 days old and doesn't have a PR open to merge it into main yet. You can browse the branch directly at:

👉 [github-samples/pets-workshop @ simplify-client-app](h...

This pull request was created from Copilot chat.

---

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.