-
Notifications
You must be signed in to change notification settings - Fork 0
Merge agentic-upgrade into main #2
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
Merged
Changes from all commits
Commits
Show all changes
27 commits
Select commit
Hold shift + click to select a range
206e03a
Upgrade Tailwind CSS to v4 and migrate config, add @tailwindcss/vite …
xcodeassociated 2b9a4c6
Upgrade dependencies: Radix UI, testing libs, GraphQL, utilities, Typ…
xcodeassociated 70113d5
Downgrade Vite and TypeScript for Node 21 compatibility; revert plugi…
xcodeassociated 21b8f3d
Upgrade to Vite 7 and Node 22 compatibility; restore Vite plugin vers…
xcodeassociated 23a62fc
Add .nvmrc for Node 22.12.0
xcodeassociated 7c67709
Add node version check script, update scripts to enforce Node >=22.12…
xcodeassociated 5f72f27
Remove node version check script and restore original scripts
xcodeassociated 86ea7d5
Add setup instructions to README, add setup script, and enforce agent…
xcodeassociated 6c75c5c
Add zsh usage and nvm check instructions to AGENTS.md
xcodeassociated 5c8502e
Emphasize critical rule: must ask for permission before any commit
xcodeassociated 23759ee
Add branch management rule: avoid changes on main without permission,…
xcodeassociated 40e1093
Fix Users test: restore vi.mock for useRolesGraphql, remove noisy con…
xcodeassociated 524a925
fix linter errors
xcodeassociated 9805499
fix tests after upgrade
xcodeassociated 66479c4
fix build
xcodeassociated d89bf38
update agent instructions
xcodeassociated f5fc589
try fix docker ci
xcodeassociated 3493b88
update github CI actions
xcodeassociated 9268b9f
fix agentic issues and apollo client
xcodeassociated 5539211
use fix eslint for current config
xcodeassociated ed319b3
fix ai style hallucinations
xcodeassociated 1d6d095
tailwind migrate 3 to 4
xcodeassociated 6e0237c
use tailwind v4 vite plugin
xcodeassociated c5c2741
manual tailwind styles migration to v4
xcodeassociated ae5fba0
upgrade manually graphql apollo client version 3 to 4
xcodeassociated 4a6bb98
update readme
xcodeassociated b757cc0
minor fix
xcodeassociated File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
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
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,24 @@ | ||
| name: CI Build and Test | ||
|
|
||
| on: | ||
| push: | ||
| branches: ['**'] | ||
| pull_request: | ||
| branches: ['**'] | ||
|
|
||
| jobs: | ||
| test-and-build: | ||
| runs-on: ubuntu-latest | ||
| steps: | ||
| - uses: actions/checkout@v4 | ||
| - uses: oven-sh/setup-bun@v2 | ||
| - name: Install dependencies | ||
| run: bun install --frozen-lockfile | ||
| - name: Codegen | ||
| run: bun run generate:graphql | ||
| - name: Run the tests | ||
| run: bun run test | ||
| - name: Build | ||
| run: bun run build | ||
| env: | ||
| CI: false |
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
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
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1 @@ | ||
| 22.12.0 |
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
| Original file line number | Diff line number | Diff line change | ||||
|---|---|---|---|---|---|---|
|
|
@@ -26,16 +26,24 @@ | |||||
| ## Project Overview | ||||||
|
|
||||||
| - **Tech Stack**: React 18, TypeScript, Vite, Tailwind CSS, Redux Toolkit (with persistence), Apollo Client, Keycloak, i18next. | ||||||
| - **Package Manager**: `npm` / `bun`. All scripts are defined in `package.json` and are **compatible with `bun run <script>`**. | ||||||
| - **Package Manager**: `bun`. All scripts are defined in `package.json` and are **compatible with `bun run <script>`**. | ||||||
| - **Path Alias**: `@/*` resolves to `src/*` (defined in `vite.config.ts` & `tsconfig.json`). | ||||||
| - **Code Generation**: GraphQL types & hooks are generated via `npm run generate:graphql`. | ||||||
| - **Code Generation**: GraphQL types & hooks are generated via `bun run generate:graphql`. | ||||||
| - **Testing**: Vitest + React Testing Library + MSW for API mocking. | ||||||
| - **Formatting**: Prettier with `prettier-plugin-tailwindcss` (installed but configuration falls back to defaults). | ||||||
|
|
||||||
| --- | ||||||
|
|
||||||
| ## Common Development Commands | ||||||
|
|
||||||
| **CRITICAL:** Before running any tests or building the project, **always regenerate the GraphQL files** to ensure they are up‑to‑date. Run: | ||||||
|
|
||||||
| ```bash | ||||||
| bun run generate:graphql | ||||||
| ``` | ||||||
|
|
||||||
| This command re‑generates the GraphQL types and hooks (e.g., `src/graphql/generated.ts`). It must be executed any time the GraphQL schema changes and must be run before `bun run test` or `bun run build`. Ensure this step is included in any CI pipeline or local workflow. | ||||||
|
||||||
| This command re‑generates the GraphQL types and hooks (e.g., `src/graphql/generated.ts`). It must be executed any time the GraphQL schema changes and must be run before `bun run test` or `bun run build`. Ensure this step is included in any CI pipeline or local workflow. | |
| This command re‑generates the GraphQL types and hooks (e.g., files in `src/graphql/generated/`). It must be executed any time the GraphQL schema changes and must be run before `bun run test` or `bun run build`. Ensure this step is included in any CI pipeline or local workflow. |
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
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
Binary file not shown.
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
Oops, something went wrong.
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
no-unused-disableis not a core ESLint rule, and there’s no plugin configured that provides it. ESLint will typically error on unknown rule names, which would breakbun run lint/CI. Remove this rule entry or replace it with the correct rule from an installed plugin (or rely on--report-unused-disable-directiveswhich is already passed on the CLI).