Skip to content

Integrate ActiveStorage feature into devcontainer#150

Open
Pjrich1313 wants to merge 64 commits into
agentsmd:dkundel/logo-handlingfrom
Pjrich1313:patch-1
Open

Integrate ActiveStorage feature into devcontainer#150
Pjrich1313 wants to merge 64 commits into
agentsmd:dkundel/logo-handlingfrom
Pjrich1313:patch-1

Conversation

@Pjrich1313

Copy link
Copy Markdown

No description provided.

dkundel-openai and others added 24 commits October 14, 2025 12:44
* 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>
@vercel

vercel Bot commented Feb 9, 2026

Copy link
Copy Markdown

@Pjrich1313 is attempting to deploy a commit to the openai Team on Vercel.

A member of the Team first needs to authorize it.

@Pjrich1313 Pjrich1313 left a comment

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot review and proceed

Pjrich1313 and others added 4 commits February 9, 2026 17:07
Co-authored-by: Pjrich1313 <129231067+Pjrich1313@users.noreply.github.com>
Co-authored-by: Pjrich1313 <129231067+Pjrich1313@users.noreply.github.com>

@Pjrich1313 Pjrich1313 left a comment

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Allow deployment of containers

@Pjrich1313 Pjrich1313 left a comment

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Load diff and batch

Copilot AI review requested due to automatic review settings June 5, 2026 16:38
@vercel

vercel Bot commented Jun 5, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
agents-md Error Error Jun 5, 2026 4:38pm
agents-md-test Error Error Jun 5, 2026 4:38pm

Request Review

@vercel

vercel Bot commented Jun 5, 2026

Copy link
Copy Markdown

Deployment failed with the following error:

Invalid vercel.json file provided

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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.

Comment thread vercel.json
@@ -0,0 +1 @@
{"$schema":"https://openapi.vercel.sh/vercel.json","github":{"enabled":true} No newline at end of file
Comment thread eslint.config.mjs
Comment on lines +1 to +7
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']),
]);
Comment thread package.json
Comment on lines +9 to +10
"lint": "eslint .",
"bash-version": "bash --version"
Comment on lines +17 to +28
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(
Comment thread batch_source.py
Comment on lines +28 to +29
default="**/*",
help="Glob pattern used to select source files (default: **/*).",
Comment thread batch_source.py
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")
Comment thread agents.md
Comment on lines +1 to +4
# AdvancedSecurity-Codeql-Init@1

inputs:
languages: 'cpp, java, python' # Validation limitations: Ensure that these languages are properly configured and supported by the runner.
Comment thread CHANGES.md
Comment on lines +21 to +22
- 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`)
Comment on lines +261 to 262
// eslint-disable-next-line react-hooks/set-state-in-effect
setShuffledAgents(shuffleAgents(agents));
Comment on lines +267 to 268
// eslint-disable-next-line react-hooks/set-state-in-effect
setIsInView(false);
@Pjrich1313 Pjrich1313 changed the base branch from main to dkundel/logo-handling June 6, 2026 00:35

@Pjrich1313 Pjrich1313 left a comment

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot make changes

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.