Integrate ActiveStorage feature into devcontainer#150
Conversation
* feat: better handle the growing list of editors * fix build * fix padding for scrolling list
The old link has been moved and now displays a 404 Not Found error. Co-authored-by: Romain Huet <romain@openai.com>
Add goose (https://github.com/block/goose) to the list of compatible AI coding agents.
Co-authored-by: Pjrich1313 <129231067+Pjrich1313@users.noreply.github.com>
|
@Pjrich1313 is attempting to deploy a commit to the openai Team on Vercel. A member of the Team first needs to authorize it. |
Pjrich1313
left a comment
There was a problem hiding this comment.
Copilot review and proceed
Co-authored-by: Pjrich1313 <129231067+Pjrich1313@users.noreply.github.com>
Co-authored-by: Pjrich1313 <129231067+Pjrich1313@users.noreply.github.com>
Header and fix #8 commm
Add to main
Pjrich1313
left a comment
There was a problem hiding this comment.
Allow deployment of containers
….py and all source files
Merge main: restore batch_source.py and missing project files
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
Deployment failed with the following error: |
There was a problem hiding this comment.
Pull request overview
Note
Copilot was unable to run its full agentic suite in this review.
This PR adds deployment/config scaffolding (Vercel + env examples), introduces linting configuration for a Next.js app, and adds a couple of Python helper/example scripts with accompanying docs.
Changes:
- Add Vercel, ESLint, devcontainer, and GitHub Actions workflow configuration files.
- Add Python scripts: a realtime API streaming example and a batch source processing helper (documented in README).
- Update Next.js dependency version and TypeScript route type import path; expand UI FAQ content.
Reviewed changes
Copilot reviewed 14 out of 17 changed files in this pull request and generated 10 comments.
Show a summary per file
| File | Description |
|---|---|
| vercel.json | Adds Vercel project configuration. |
| realtime_example.py | Adds an OpenAI streaming example script. |
| package.json | Updates Next.js version and switches linting to ESLint; adds related deps. |
| next-env.d.ts | Adjusts generated routes type import path. |
| eslint.config.mjs | Adds ESLint flat config for Next core-web-vitals. |
| components/FAQSection.tsx | Adds a new FAQ entry with guidance for ambiguous repo-only requests. |
| components/CompatibilitySection.tsx | Disables a React Hooks lint rule around state updates in effects. |
| batch_source.py | Adds a batching/merge helper script for text output generation. |
| agents.md | Adds agent/config-like content in a markdown file. |
| README.md | Documents how to run batch_source.py. |
| CHANGES.md | Adds a change log / consolidation summary. |
| .gitignore | Adds npm lockfile ignore and Python ignore patterns (duplicated). |
| .github/workflows/webpack.yml | Adds a Node/Webpack GitHub Actions workflow. |
| .eslintrc.json | Adds legacy ESLint config extending Next core-web-vitals. |
| .env.example | Adds environment variable template. |
| .devcontainer/devcontainer.json | Adds a devcontainer definition. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| @@ -0,0 +1 @@ | |||
| {"$schema":"https://openapi.vercel.sh/vercel.json","github":{"enabled":true} No newline at end of file | |||
| import { defineConfig, globalIgnores } from 'eslint/config'; | ||
| import nextVitals from 'eslint-config-next/core-web-vitals'; | ||
|
|
||
| export default defineConfig([ | ||
| ...nextVitals, | ||
| globalIgnores(['.next/**', 'out/**', 'build/**', 'next-env.d.ts']), | ||
| ]); |
| "lint": "eslint .", | ||
| "bash-version": "bash --version" |
| steps: step new version | ||
| - uses: actions/checkout@v4 | ||
|
|
||
| - name: Use Node.js ${{ matrix.node-version }} | ||
| uses: actions/setup-node@v4 | ||
| with: | ||
| node-version: ${{ matrix.node-version }} | ||
|
|
||
| - name: Build | ||
| run: | | ||
| npm install | ||
| npx webpack( |
| default="**/*", | ||
| help="Glob pattern used to select source files (default: **/*).", |
| with out_path.open("w", encoding="utf-8") as out: | ||
| for source_file in batch: | ||
| relative = source_file.relative_to(source_root) | ||
| content = source_file.read_text(encoding="utf-8") |
| # AdvancedSecurity-Codeql-Init@1 | ||
|
|
||
| inputs: | ||
| languages: 'cpp, java, python' # Validation limitations: Ensure that these languages are properly configured and supported by the runner. |
| - Added `.eslintrc.json` with `next/core-web-vitals` preset for consistent linting | ||
| - Fixed `next-env.d.ts` type import path (`types/routes.d.ts` → `dev/types/routes.d.ts`) |
| // eslint-disable-next-line react-hooks/set-state-in-effect | ||
| setShuffledAgents(shuffleAgents(agents)); |
| // eslint-disable-next-line react-hooks/set-state-in-effect | ||
| setIsInView(false); |
No description provided.