Skip to content

Staging version - #236

Merged
bourgeoa merged 2 commits into
stagingfrom
staging-version
Aug 20, 2026
Merged

Staging version#236
bourgeoa merged 2 commits into
stagingfrom
staging-version

Conversation

@bourgeoa

Copy link
Copy Markdown
Contributor

No description provided.

Copilot AI lite review requested due to automatic review settings August 20, 2026 20:19
@bourgeoa
bourgeoa changed the base branch from main to staging August 20, 2026 20:20
@bourgeoa
bourgeoa merged commit ea8fcf5 into staging Aug 20, 2026
1 check passed

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This PR migrates the project from a Webpack/Babel-based build and local dev harness to a Vite + solidos-toolkit setup, and updates the package’s published entrypoints for a staging/prerelease version line.

Changes:

  • Replace Webpack/Babel build + dev server configuration with a Vite config using solidos-toolkit.
  • Update build output conventions (move from lib/ to dist/) and switch package.json entrypoints/exports accordingly, with a prerelease version bump.
  • Adjust folder-pane drag/drop upload target area styling/behavior and remove the old dev/ harness.

Reviewed changes

Copilot reviewed 1 out of 2 changed files in this pull request and generated no comments.

Show a summary per file
File Description
webpack.module.rules.mjs Removed Webpack module rules as part of the migration away from Webpack.
webpack.dev.config.mjs Removed Webpack dev-server configuration.
webpack.config.mjs Removed production Webpack build configuration (incl. minified build path).
vite.config.mts Added Vite build configuration via solidos-toolkit for the new build pipeline.
tsconfig.json Updated TS output/exclude paths and module resolution mode for bundler-oriented workflows.
src/styles/folderPane.css Added .folderPaneDropZone styling for the updated drop/upload area.
src/index.ts Added a library entry module for the new build entrypoint.
src/folderPane.ts Updated drop target wiring to attach drag/drop to the new drop zone element.
package.json Updated version, build scripts, entrypoints/exports, and dependencies for Vite-based build/publish.
dev/index.js Removed legacy standalone dev harness JS.
dev/index.html Removed legacy standalone dev harness HTML.
dev/dev-global.css Removed legacy dev-only global CSS.
dev/context.js Removed legacy dev-only context mock.
babel.config.mjs Removed Babel configuration (no longer needed with Vite pipeline).
.gitignore Updated ignored build output directory from lib to dist.
.github/workflows/ci.yml Updated CI triggers for staging and added dependency-update + prerelease publish workflow usage.
Suppressed comments (1)

package.json:76

  • vite, vitest, and vitest-axe are listed under runtime dependencies, but they appear to be build/test tooling only (no runtime imports in src/). Keeping these as runtime deps forces downstream consumers to install large toolchains unnecessarily. Consider moving them to devDependencies instead.
  "dependencies": {
    "rdflib": "^2.4.0",
    "solid-logic": "^6.0.0-0",
    "solid-ui": "^5.0.0-0",
    "vite": "^8.0.16",
    "vitest": "^4.0.18",
    "vitest-axe": "^0.1.0"
  },

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

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.

2 participants