Add Sonoxo Agentic Data Cloud - #37
Conversation
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: fbacae48fa
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| import hostingConfig from "./.openai/hosting.json"; | ||
| import { sites } from "./build/sites-vite-plugin"; |
There was a problem hiding this comment.
Commit the Vite inputs needed by build and dev
In a fresh checkout, this configuration cannot be loaded because neither ./.openai/hosting.json nor ./build/sites-vite-plugin is tracked; a repo-wide rg --files search found no equivalent files or generated artifacts. Consequently the vite development command and the Vinext build invoked by package.json fail on unresolved local imports before compiling the application.
Useful? React with 👍 / 👎.
| "dev": "WRANGLER_LOG_PATH=.wrangler/wrangler.log vite", | ||
| "build": "bash scripts/build-verified.sh", | ||
| "start": "WRANGLER_LOG_PATH=.wrangler/wrangler.log vinext start", | ||
| "test": "npm run build && node --test tests/rendered-html.test.mjs", |
There was a problem hiding this comment.
Add the test file referenced by the test script
Once the build failure is repaired, npm test still cannot succeed because tests/rendered-html.test.mjs is not present anywhere in the commit or repository. In the supported Node 24 environment, invoking this exact node --test suffix exits with status 1 and Could not find 'tests/rendered-html.test.mjs', so the declared verification command is permanently broken.
Useful? React with 👍 / 👎.
| "build": "bash scripts/build-verified.sh", | ||
| "start": "WRANGLER_LOG_PATH=.wrangler/wrangler.log vinext start", | ||
| "test": "npm run build && node --test tests/rendered-html.test.mjs", | ||
| "lint": "bash scripts/sites-env.sh -- eslint . --ignore-pattern dist --ignore-pattern .next", |
There was a problem hiding this comment.
Provide an ESLint flat configuration
This app pins ESLint 9.39.4 but does not track an eslint.config.js, .mjs, or .cjs anywhere in the repository, so npm run lint exits before checking any files. ESLint's diagnostic confirms the relevant behavior: “From ESLint v9.0.0, the default configuration file is now eslint.config.js.”
Useful? React with 👍 / 👎.
Sonoxo Agentic Data Cloud
Adds a production-built agentic social data warehouse application under
apps/sonoxo-agentic-cloud.Included
Verification
npm run db:generatepassednpm run buildpassed