feature/IO-3725-RPS-Changes - Add AI context files, fix auto start dev tools in local
This commit is contained in:
105
.ai/recent-changes.md
Normal file
105
.ai/recent-changes.md
Normal file
@@ -0,0 +1,105 @@
|
||||
# Recent Changes and Migration Context
|
||||
|
||||
This file captures recent cleanup/migration work so future agents do not reverse it.
|
||||
|
||||
## AntD 5 Deprecation Cleanup
|
||||
|
||||
Warnings addressed:
|
||||
|
||||
- `Collapse expandIconPosition` left/right deprecated.
|
||||
- `rc-collapse children` deprecated.
|
||||
- `Table rowKey` index parameter deprecated.
|
||||
- Static `message` cannot consume dynamic theme context.
|
||||
- `Dropdown overlay` deprecated.
|
||||
- `RangePicker ranges` deprecated.
|
||||
|
||||
Patterns now used:
|
||||
|
||||
- `Collapse items`
|
||||
- `expandIconPosition="end"`
|
||||
- `Dropdown menu`
|
||||
- stable table row keys
|
||||
- `RangePicker presets`
|
||||
- `src/util/antdFeedback.js` for feedback APIs
|
||||
- AntD `<App>` wrapper and bridge in `src/App/App.jsx`
|
||||
|
||||
## Redux CreateStore Warning
|
||||
|
||||
`src/redux/store.js` imports:
|
||||
|
||||
```js
|
||||
import { legacy_createStore as createStore, applyMiddleware, compose } from "redux";
|
||||
```
|
||||
|
||||
This intentionally keeps the current Redux architecture while avoiding the visual deprecation warning. Do not change to plain `createStore`.
|
||||
|
||||
## Job Detail Flash Fix
|
||||
|
||||
`src/components/organisms/jobs-detail/jobs-detail.organism.jsx` debounces selected job id and shows a skeleton during transitions. This prevents a fraction-of-a-second stale/partial render in the left panel.
|
||||
|
||||
When editing this area:
|
||||
|
||||
- Preserve stale-data guards.
|
||||
- Preserve a stable loading state when selected id changes.
|
||||
- Do not render details for a job whose id does not match current selection.
|
||||
|
||||
## Apollo Pagination/Cache Fixes
|
||||
|
||||
`src/graphql/GraphQLClient.js` includes offset merge policies:
|
||||
|
||||
- `Query.jobs`
|
||||
- `Query.search_jobs`
|
||||
|
||||
List components were hardened against missing previous/incoming arrays.
|
||||
|
||||
## Electron Package Upgrades
|
||||
|
||||
Packages upgraded together:
|
||||
|
||||
- `electron-context-menu` to `^4.1.2`
|
||||
- `electron-is-dev` to `^3.0.1`
|
||||
- `electron-store` to `^11.0.2`
|
||||
- `electron` to `^42.3.0`
|
||||
- `electron-builder` to `^26.8.1`
|
||||
- `@sentry/electron` to `^7.13.0`
|
||||
|
||||
Important migration consequences:
|
||||
|
||||
- `electron-store` is initialized asynchronously via dynamic import.
|
||||
- `electron-context-menu` is dynamically imported.
|
||||
- `electron-is-dev` is no longer used in runtime code; use `app.isPackaged`/process checks.
|
||||
- Sentry uses protocol IPC mode.
|
||||
- Electron builder pack check uses `electron-builder.pack-check.cjs`.
|
||||
|
||||
## Electron Warning Cleanup
|
||||
|
||||
Circular dependency warning fixed by removing `mainWindow` export/import patterns. IPC modules use Electron APIs directly:
|
||||
|
||||
```js
|
||||
BrowserWindow.fromWebContents(event.sender)
|
||||
```
|
||||
|
||||
Sentry preload deprecation warnings were addressed by using protocol IPC mode:
|
||||
|
||||
```js
|
||||
ipcMode: Sentry.IPCMode.Protocol
|
||||
```
|
||||
|
||||
DevTools behavior:
|
||||
|
||||
- Development opens DevTools by default again.
|
||||
- Use `ELECTRON_OPEN_DEVTOOLS=0` to suppress.
|
||||
- React DevTools extension install remains opt-in.
|
||||
|
||||
## Pack Check Safety
|
||||
|
||||
User specifically requested pack checks without publishing.
|
||||
|
||||
`package.json`:
|
||||
|
||||
```json
|
||||
"pack": "electron-builder --dir --publish never --config electron-builder.pack-check.cjs"
|
||||
```
|
||||
|
||||
`electron-builder.pack-check.cjs` disables Azure signing options for this local check. Do not remove `--publish never` from the pack script.
|
||||
|
||||
Reference in New Issue
Block a user