Drop editor in favor of ord-app

[skearnes]

Summary

• Removes the Flask-based editor under ord_interface/editor/ — its job is now handled by the standalone ord-app, which the Contribute link in the Vue header already points to.
• Tears out everything that existed only to support the editor: the interface.py Flask entrypoint, the flask upstream + /editor/ location in nginx, the flask gunicorn worker, the closure-library/protoc/closure-compiler/jquery-externs build chain in the Dockerfile, the editor schema/migrate step in build_test_database.sh, the node editor/js/test.js JS tests, the --ignore=ord_interface/editor pytest flag and puppeteer install in CI, and the flask/pygithub deps in setup.py.
• Drops the orphaned app/src/views/contribute/ Vue views (already not wired into the router) and the orphaned static about.html (Vue's About.vue serves /about).
• Switches the AWS Copilot front-end healthcheck from /editor/healthcheck to /, drops the now-unused GH_CLIENT_ID/GH_CLIENT_SECRET secrets there and in docker-compose.yml.
• Adds an nginx 301 from /editor (and /editor/*) to https://app.open-reaction-database.org/ so old bookmarks land on the new editor instead of a blank SPA shell.
• Bumps the test_app CI runner from ubuntu-22.04 to ubuntu-latest now that puppeteer is gone.
• Removes a stray dump.rdb Redis dump from the repo root and adds it to .gitignore.

Follow-ups (out of scope for this PR)

• Delete the now-unused SSM parameters under /copilot/ord-editor/secrets/ (GH_CLIENT_ID, GH_CLIENT_SECRET) and the GitHub OAuth app behind them.
• Consider switching the Copilot healthcheck from / to a FastAPI route (e.g. /api/datasets) so a backend outage actually fails the health check; currently / is green even if FastAPI is down.

Test plan

• ./run_tests.sh builds the slimmed-down image and the pytest suite passes — verified locally, 39/39 passed
• Docker image starts cleanly (no flask socket; only fastapi behind nginx) — only /run/fastapi.sock exists; gunicorn runs ord_interface.api.main:app only
• Vue SPA loads, /browse, /search, /about, /ketcher all work — all return HTTP 200 via nginx
• Header Contribute link still points to app.open-reaction-database.org — confirmed at app/src/components/HeaderNav.vue:39
• Copilot front-end healthcheck at / returns 200 — confirmed via curl -I http://localhost:8080/
• /editor, /editor/, /editor/foo, /editor/healthcheck all return 301 → https://app.open-reaction-database.org/; /editorx falls through to the SPA

🤖 Generated with Claude Code

[bdeadman]

@skearnes - yes we should do this to clean up the code, but I hesitate to rip the old editor out of the production instance without communicating a clear timeline for the user community to remove any important data. The latest statement we have made on the old editor is from the announcement of the new editor here https://www.linkedin.com/pulse/next-generation-reaction-editor-here-open-reaction-database-tfste.

[bdeadman]

My understanding is that this 'about' file no longer needed since the website uses this file (https://github.com/open-reaction-database/ord-interface/blob/8731895b44a960f0448a72042fa9981df2846c22/app/src/views/About.vue) instead for the about page.

[bdeadman]

Happy to approve the code changes as they are, but I'd like to have a conversation about how we manage the retirement of the legacy editor from the production instance.