Open
Conversation
Contributor
There was a problem hiding this comment.
Pull request overview
Migrates the JavaScript and AssemblyScript example apps out of this repository into their respective SDK repositories, leaving behind lightweight READMEs that link to the new locations. Also updates the npm lockfile used by the Rust custom_error_pages example’s frontend tooling.
Changes:
- Remove in-repo JavaScript examples (template invoice, AB testing, static assets, react app, MCP server, KV store, geo redirect) and associated build scripts/configs.
- Remove in-repo AssemblyScript examples and build config, replacing with pointers to
proxy-wasm-sdk-as. - Update docs (JS/AS READMEs) to link to the new repos and refresh build/run instructions; bump some npm dependencies in
custom_error_pageslockfile.
Reviewed changes
Copilot reviewed 114 out of 166 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| rust/cdn/custom_error_pages/package-lock.json | Adds lockfile metadata and updates dependency resolutions for the example’s JS toolchain. |
| javascript/src/template-invoice/logo.js | Removed (example migrated). |
| javascript/src/template-invoice/invoice.png | Removed (example migrated). |
| javascript/src/template-invoice/index.js | Removed (example migrated). |
| javascript/src/template-invoice/htmlTemplate.js | Removed (example migrated). |
| javascript/src/template-invoice/cssStyles.js | Removed (example migrated). |
| javascript/src/template-invoice/README.md | Removed (example migrated). |
| javascript/src/template-invoice-ab-testing/index.js | Removed (example migrated). |
| javascript/src/template-invoice-ab-testing/htmlTemplate.js | Removed (example migrated). |
| javascript/src/template-invoice-ab-testing/cssStyles.js | Removed (example migrated). |
| javascript/src/template-invoice-ab-testing/README.md | Removed (example migrated). |
| javascript/src/static-assets/tsconfig.json | Removed (example migrated). |
| javascript/src/static-assets/templates/index.html | Removed (example migrated). |
| javascript/src/static-assets/styles/index.css | Removed (example migrated). |
| javascript/src/static-assets/src/templates-static-assets.ts | Removed (example migrated). |
| javascript/src/static-assets/src/styles-static-assets.ts | Removed (example migrated). |
| javascript/src/static-assets/src/jsx-page.tsx | Removed (example migrated). |
| javascript/src/static-assets/src/index.tsx | Removed (example migrated). |
| javascript/src/static-assets/src/images-static-assets.ts | Removed (example migrated). |
| javascript/src/static-assets/package.json | Removed (example migrated). |
| javascript/src/static-assets/images/home.png | Removed (example migrated). |
| javascript/src/static-assets/images/gcore.png | Removed (example migrated). |
| javascript/src/static-assets/README.md | Removed (example migrated). |
| javascript/src/react-app/tsconfig.json | Removed (example migrated). |
| javascript/src/react-app/src/utils/index.ts | Removed (example migrated). |
| javascript/src/react-app/src/types/styled.d.ts | Removed (example migrated). |
| javascript/src/react-app/src/types/index.ts | Removed (example migrated). |
| javascript/src/react-app/src/styles/theme.ts | Removed (example migrated). |
| javascript/src/react-app/src/styles/globalStyles.ts | Removed (example migrated). |
| javascript/src/react-app/src/services/weatherService.ts | Removed (example migrated). |
| javascript/src/react-app/src/services/types.ts | Removed (example migrated). |
| javascript/src/react-app/src/provider/AppStateContext/types.ts | Removed (example migrated). |
| javascript/src/react-app/src/provider/AppStateContext/index.tsx | Removed (example migrated). |
| javascript/src/react-app/src/index.jsx | Removed (example migrated). |
| javascript/src/react-app/src/index.css | Removed (example migrated). |
| javascript/src/react-app/src/hooks/useTemp.ts | Removed (example migrated). |
| javascript/src/react-app/src/config/index.ts | Removed (example migrated). |
| javascript/src/react-app/src/components/index.ts | Removed (example migrated). |
| javascript/src/react-app/src/components/WeeklyForecast/styled.ts | Removed (example migrated). |
| javascript/src/react-app/src/components/WeeklyForecast/index.tsx | Removed (example migrated). |
| javascript/src/react-app/src/components/Weather/styled.ts | Removed (example migrated). |
| javascript/src/react-app/src/components/Weather/index.tsx | Removed (example migrated). |
| javascript/src/react-app/src/components/Temperature/index.tsx | Removed (example migrated). |
| javascript/src/react-app/src/components/Loader/styled.ts | Removed (example migrated). |
| javascript/src/react-app/src/components/Loader/index.tsx | Removed (example migrated). |
| javascript/src/react-app/src/components/Highlights/styled.ts | Removed (example migrated). |
| javascript/src/react-app/src/components/Highlights/index.tsx | Removed (example migrated). |
| javascript/src/react-app/src/components/Header/styled.ts | Removed (example migrated). |
| javascript/src/react-app/src/components/Header/index.tsx | Removed (example migrated). |
| javascript/src/react-app/src/components/CityInput/styled.ts | Removed (example migrated). |
| javascript/src/react-app/src/components/CityInput/index.tsx | Removed (example migrated). |
| javascript/src/react-app/src/components/Card/types.ts | Removed (example migrated). |
| javascript/src/react-app/src/components/Card/styled.ts | Removed (example migrated). |
| javascript/src/react-app/src/components/Card/index.tsx | Removed (example migrated). |
| javascript/src/react-app/src/App.tsx | Removed (example migrated). |
| javascript/src/react-app/src/App.styled.tsx | Removed (example migrated). |
| javascript/src/react-app/public/weather_icons/wind-night.png | Removed (example migrated). |
| javascript/src/react-app/public/weather_icons/wind-day.png | Removed (example migrated). |
| javascript/src/react-app/public/weather_icons/uv.png | Removed (example migrated). |
| javascript/src/react-app/public/weather_icons/sunset.png | Removed (example migrated). |
| javascript/src/react-app/public/weather_icons/sunrise.png | Removed (example migrated). |
| javascript/src/react-app/public/weather_icons/pressure.png | Removed (example migrated). |
| javascript/src/react-app/public/weather_icons/humidity.png | Removed (example migrated). |
| javascript/src/react-app/public/weather_icons/clouds.png | Removed (example migrated). |
| javascript/src/react-app/public/weather_icons/50n.png | Removed (example migrated). |
| javascript/src/react-app/public/weather_icons/50d.png | Removed (example migrated). |
| javascript/src/react-app/public/weather_icons/13n.png | Removed (example migrated). |
| javascript/src/react-app/public/weather_icons/13d.png | Removed (example migrated). |
| javascript/src/react-app/public/weather_icons/11n.png | Removed (example migrated). |
| javascript/src/react-app/public/weather_icons/11d.png | Removed (example migrated). |
| javascript/src/react-app/public/weather_icons/10n.png | Removed (example migrated). |
| javascript/src/react-app/public/weather_icons/10d.png | Removed (example migrated). |
| javascript/src/react-app/public/weather_icons/09n.png | Removed (example migrated). |
| javascript/src/react-app/public/weather_icons/09d.png | Removed (example migrated). |
| javascript/src/react-app/public/weather_icons/04n.png | Removed (example migrated). |
| javascript/src/react-app/public/weather_icons/04d.png | Removed (example migrated). |
| javascript/src/react-app/public/weather_icons/03n.png | Removed (example migrated). |
| javascript/src/react-app/public/weather_icons/03d.png | Removed (example migrated). |
| javascript/src/react-app/public/weather_icons/02n.png | Removed (example migrated). |
| javascript/src/react-app/public/weather_icons/02d.png | Removed (example migrated). |
| javascript/src/react-app/public/weather_icons/01n.png | Removed (example migrated). |
| javascript/src/react-app/public/weather_icons/01d.png | Removed (example migrated). |
| javascript/src/react-app/public/robots.txt | Removed (example migrated). |
| javascript/src/react-app/public/manifest.json | Removed (example migrated). |
| javascript/src/react-app/public/logo512.png | Removed (example migrated). |
| javascript/src/react-app/public/logo192.png | Removed (example migrated). |
| javascript/src/react-app/public/index.html | Removed (example migrated). |
| javascript/src/react-app/public/favicon.ico | Removed (example migrated). |
| javascript/src/react-app/package.json | Removed (example migrated). |
| javascript/src/react-app/images/preview.png | Removed (example migrated). |
| javascript/src/react-app/images/apikey.png | Removed (example migrated). |
| javascript/src/react-app/fastedge-server/middleware.ts | Removed (example migrated). |
| javascript/src/react-app/fastedge-server/index.ts | Removed (example migrated). |
| javascript/src/react-app/fastedge-server/forecast.ts | Removed (example migrated). |
| javascript/src/react-app/fastedge-server/currentWeather.ts | Removed (example migrated). |
| javascript/src/react-app/esbuild/server.js | Removed (example migrated). |
| javascript/src/react-app/craco.config.cjs | Removed (example migrated). |
| javascript/src/react-app/README.md | Removed (example migrated). |
| javascript/src/react-app/.gitignore | Removed (example migrated). |
| javascript/src/react-app/.fastedge/static-index.js | Removed (example migrated). |
| javascript/src/react-app/.fastedge/package.json | Removed (example migrated). |
| javascript/src/react-app/.fastedge/jsconfig.json | Removed (example migrated). |
| javascript/src/react-app/.fastedge/build-config.js | Removed (example migrated). |
| javascript/src/react-app/.eslintrc | Removed (example migrated). |
| javascript/src/mcp-server/tsconfig.json | Removed (example migrated). |
| javascript/src/mcp-server/src/types.ts | Removed (example migrated). |
| javascript/src/mcp-server/src/server.ts | Removed (example migrated). |
| javascript/src/mcp-server/src/index.ts | Removed (example migrated). |
| javascript/src/mcp-server/package.json | Removed (example migrated). |
| javascript/src/mcp-server/images/mcp.jpg | Removed (example migrated). |
| javascript/src/mcp-server/images/github-settings.jpg | Removed (example migrated). |
| javascript/src/mcp-server/images/get-weather.jpg | Removed (example migrated). |
| javascript/src/mcp-server/images/get-weather-response.jpg | Removed (example migrated). |
| javascript/src/mcp-server/images/get-alerts.jpg | Removed (example migrated). |
| javascript/src/mcp-server/images/get-alerts-response.jpg | Removed (example migrated). |
| javascript/src/mcp-server/images/agent-mode.jpg | Removed (example migrated). |
| javascript/src/mcp-server/README.md | Removed (example migrated). |
| javascript/src/mcp-server/LICENSE | Removed (example migrated). |
| javascript/src/mcp-server/.vscode/mcp.json | Removed (example migrated). |
| javascript/src/mcp-server/.gitignore | Removed (example migrated). |
| javascript/src/kv-store/tsconfig.json | Removed (example migrated). |
| javascript/src/kv-store/src/utils.ts | Removed (example migrated). |
| javascript/src/kv-store/src/index.ts | Removed (example migrated). |
| javascript/src/kv-store/README.md | Removed (example migrated). |
| javascript/src/kv-store/.fastedge/build-config.js | Removed (example migrated). |
| javascript/src/geo-redirect/index.js | Removed (example migrated). |
| javascript/src/geo-redirect/env-vars.png | Removed (example migrated). |
| javascript/src/geo-redirect/README.md | Removed (example migrated). |
| javascript/src/ab-testing/index.js | Removed (example migrated). |
| javascript/src/ab-testing/README.md | Removed (example migrated). |
| javascript/scripts/serve-wasm.js | Removed (example migrated). |
| javascript/scripts/build.js | Removed (example migrated). |
| javascript/package.json | Removed (example migrated). |
| javascript/README.md | Updated to point to FastEdge-sdk-js examples and new build/run instructions. |
| javascript/.gitignore | Removed (no longer needed after example migration). |
| assemblyscript/package.json | Removed (examples migrated). |
| assemblyscript/package-lock.json | Removed (examples migrated). |
| assemblyscript/assembly/tsconfig.json | Removed (examples migrated). |
| assemblyscript/assembly/properties/index.ts | Removed (examples migrated). |
| assemblyscript/assembly/properties/README.md | Removed (examples migrated). |
| assemblyscript/assembly/logTime/index.ts | Removed (examples migrated). |
| assemblyscript/assembly/logTime/README.md | Removed (examples migrated). |
| assemblyscript/assembly/kvStore/utils.ts | Removed (examples migrated). |
| assemblyscript/assembly/kvStore/index.ts | Removed (examples migrated). |
| assemblyscript/assembly/kvStore/README.md | Removed (examples migrated). |
| assemblyscript/assembly/jwt/secret-vars.png | Removed (examples migrated). |
| assemblyscript/assembly/jwt/index.ts | Removed (examples migrated). |
| assemblyscript/assembly/jwt/README.md | Removed (examples migrated). |
| assemblyscript/assembly/headers/index.ts | Removed (examples migrated). |
| assemblyscript/assembly/headers/README.md | Removed (examples migrated). |
| assemblyscript/assembly/geoRedirect/index.ts | Removed (examples migrated). |
| assemblyscript/assembly/geoRedirect/env_vars.png | Removed (examples migrated). |
| assemblyscript/assembly/geoRedirect/README.md | Removed (examples migrated). |
| assemblyscript/assembly/geoBlock/index.ts | Removed (examples migrated). |
| assemblyscript/assembly/geoBlock/env-vars.png | Removed (examples migrated). |
| assemblyscript/assembly/geoBlock/README.md | Removed (examples migrated). |
| assemblyscript/assembly/body/index.ts | Removed (examples migrated). |
| assemblyscript/assembly/body/README.md | Removed (examples migrated). |
| assemblyscript/asconfig.json | Removed (examples migrated). |
| assemblyscript/README.md | Updated to point to proxy-wasm-sdk-as examples and new build/run instructions. |
| assemblyscript/.gitignore | Removed (no longer needed after example migration). |
Files not reviewed (2)
- assemblyscript/package-lock.json: Language not supported
- rust/cdn/custom_error_pages/package-lock.json: Language not supported
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
qrdl
requested changes
Mar 10, 2026
Collaborator
qrdl
left a comment
There was a problem hiding this comment.
Please mention in README that examples for JS and AS were moved to SDK repos (with direct links). We will keep this repo alive for some time because it is mentioned in the docs
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.