Merge pull request #96 from KubrickCode/develop/shlee/76

refactor: flatten project structure to src-level organization

Author: KubrickCode

.github/workflows/lint.yml

@@ -9,7 +9,7 @@ on:
       - unlocked
     paths:
       - "src/extension/**"
-      - "src/web-view/**"
+      - "src/view/**"
       - "**/*.json"
       - "**/*.yml"
       - "**/*.yaml"
@@ -19,7 +19,7 @@ on:
       - main
     paths:
       - "src/extension/**"
-      - "src/web-view/**"
+      - "src/view/**"
       - "**/*.json"
       - "**/*.yml"
       - "**/*.yaml"
@@ -52,8 +52,8 @@ jobs:
       - name: Lint Extension
         run: just lint extension
 
-  lint-web-view:
-    name: Lint Web View Code
+  lint-view:
+    name: Lint View Code
     runs-on: ubuntu-latest
     steps:
       - name: Checkout
@@ -74,8 +74,8 @@ jobs:
       - name: Install dependencies
         run: just deps
 
-      - name: Lint Web View
-        run: just lint web-view
+      - name: Lint View
+        run: just lint view
 
   lint-config:
     name: Lint Config Files

.lintstagedrc.js

@@ -1,6 +1,6 @@
 module.exports = {
   "src/extension/**/*.ts": () => "just lint extension",
-  "src/web-view/**/*.{ts,tsx}": () => "just lint web-view",
+  "src/view/**/*.{ts,tsx}": () => "just lint view",
   "**/*.{json,yml,yaml,md}": () => "just lint config",
   justfile: () => "just lint justfile",
 };

.vscodeignore

@@ -1,13 +1,21 @@
 # Source code files - only include compiled output
-src/extension/src/
+src/extension/main.ts
 src/extension/__mocks__/
 src/extension/*.test.ts
 src/extension/*.test.js
 src/extension/*.spec.ts
 src/extension/*.spec.js
+src/internal/
+src/pkg/
+src/shared/
+src/tests/
 
 # Development dependencies
-src/web-view/node_modules/
+src/view/node_modules/
+src/node_modules/
+src/package.json
+src/pnpm-lock.yaml
+src/eslint.config.js
 
 # Exclude dev dependencies from extension package
 src/extension/node_modules/@jest/
@@ -30,16 +38,8 @@ src/extension/.gitignore
 src/extension/yarn.lock
 src/extension/package-lock.json
 
-# Web view source (keep only built output)
-src/web-view/src/
-src/web-view/public/
-src/web-view/node_modules/
-src/web-view/package.json
-src/web-view/package-lock.json
-src/web-view/yarn.lock
-src/web-view/tsconfig.json
-src/web-view/vite.config.ts
-src/web-view/.gitignore
+# View source (keep only built output in view-dist)
+src/view/
 
 # Root public folder - exclude large screenshots but keep logo
 public/screenshots/
@@ -48,6 +48,7 @@ public/*.gif
 
 # Root level dependencies
 node_modules/
+.pnpm-store/
 frontend/
 
 # Development files

CLAUDE.md

@@ -61,7 +61,7 @@ npm run ovsx-publish     # Publish to Open VSX Registry
 
 ```bash
 # React + Vite configuration UI
-cd src/web-view
+cd src/view
 npm run dev      # Start dev server
 npm run build    # Build for production
 npm run lint     # Run ESLint
@@ -144,7 +144,7 @@ src/extension/src/     # TypeScript extension code
   command-executor.ts  # Command execution logic
   keyboard-layout-converter.ts  # Multi-language support
 
-src/web-view/src/      # React configuration UI
+src/view/src/          # React configuration UI
   app.tsx              # Root component
   core/                # Reusable UI components
   components/          # Feature-specific components

justfile

@@ -1,46 +1,50 @@
 set dotenv-load := true
 
 root_dir := justfile_directory()
-extension_dir := root_dir + "/src/extension"
-web_view_dir := root_dir + "/src/web-view"
+src_dir := root_dir + "/src"
+extension_dir := src_dir + "/extension"
+view_dir := src_dir + "/view"
 
-deps: deps-root deps-extension deps-web-view
+deps: deps-root deps-src deps-extension deps-view
 
 deps-root:
     cd "{{ root_dir }}" && pnpm install
 
+deps-src:
+    cd "{{ src_dir }}" && pnpm install
+
 deps-extension:
     cd "{{ extension_dir }}" && pnpm install
 
-deps-web-view:
-    cd "{{ web_view_dir }}" && pnpm install
+deps-view:
+    cd "{{ view_dir }}" && pnpm install
 
 install-package:
     cd "{{ root_dir }}" && pnpm install-package
 
 clean-build:
-    rm -rf "{{ web_view_dir }}/dist"
-    rm -rf "{{ extension_dir }}/web-view-dist"
-    rm -rf "{{ root_dir }}/out"
+    rm -rf "{{ view_dir }}/dist"
+    rm -rf "{{ extension_dir }}/view-dist"
+    rm -rf "{{ extension_dir }}/out"
 
 lint target="all":
     #!/usr/bin/env bash
     set -euox pipefail
     case "{{ target }}" in
       all)
         just lint extension
-        just lint web-view
+        just lint view
         just lint config
         just lint justfile
         ;;
       extension)
-        npx prettier --write "{{ extension_dir }}/src/**/*.ts"
-        cd "{{ extension_dir }}"
-        pnpm lint
+        npx prettier --write "{{ src_dir }}/{extension,internal,pkg,shared,tests}/**/*.ts"
+        cd "{{ src_dir }}"
+        npx eslint --fix extension/main.ts internal pkg shared tests
         ;;
-      web-view)
-        npx prettier --write "{{ web_view_dir }}/src/**/*.{ts,tsx}"
-        cd "{{ web_view_dir }}"
+      view)
+        npx prettier --write "{{ view_dir }}/src/**/*.{ts,tsx}"
+        cd "{{ view_dir }}"
         pnpm lint
         ;;
       config)
@@ -56,8 +60,8 @@ lint target="all":
     esac
 
 package: clean-build
-    cd "{{ web_view_dir }}" && pnpm build
-    cp -r "{{ web_view_dir }}/dist" "{{ extension_dir }}/web-view-dist"
+    cd "{{ view_dir }}" && pnpm build
+    cp -r "{{ view_dir }}/dist" "{{ extension_dir }}/view-dist"
     cd "{{ extension_dir }}" && pnpm compile
     cd "{{ root_dir }}" && pnpm package
 
@@ -100,7 +104,7 @@ release:
     @echo "📊 Check progress: https://github.com/KubrickCode/quick-command-buttons/actions"
 
 run-view:
-    cd "{{ web_view_dir }}" && pnpm dev
+    cd "{{ view_dir }}" && pnpm dev
 
 test mode="":
     #!/usr/bin/env bash

package.json

@@ -31,7 +31,7 @@
   "activationEvents": [
     "onStartupFinished"
   ],
-  "main": "./src/extension/out/src/main.js",
+  "main": "./src/extension/out/extension/main.js",
   "contributes": {
     "commands": [
       {

src/extension/eslint.config.js renamed to src/eslint.config.js

@@ -4,16 +4,18 @@ import perfectionist from "eslint-plugin-perfectionist";
 import tseslint from "typescript-eslint";
 
 export default [
+  {
+    ignores: ["**/out/**", "**/__mocks__/**", "**/*.config.*", "**/node_modules/**", "**/view/**", "**/coverage/**"],
+  },
   js.configs.recommended,
   ...tseslint.configs.recommended,
   {
-    files: ["src/**/*.ts", "test/**/*.ts"],
-    ignores: ["dist", "node_modules"],
+    files: ["extension/main.ts", "internal/**/*.ts", "pkg/**/*.ts", "shared/**/*.ts"],
     languageOptions: {
       parser: tseslint.parser,
       parserOptions: {
         ecmaVersion: "latest",
-        project: "./tsconfig.json",
+        project: "./extension/tsconfig.json",
         sourceType: "module",
         tsconfigRootDir: import.meta.dirname,
       },
@@ -77,4 +79,42 @@ export default [
       ],
     },
   },
+  {
+    files: ["tests/**/*.ts"],
+    languageOptions: {
+      parser: tseslint.parser,
+      parserOptions: {
+        ecmaVersion: "latest",
+        sourceType: "module",
+      },
+    },
+    plugins: {
+      "@typescript-eslint": tseslint.plugin,
+      import: importPlugin,
+      perfectionist,
+    },
+    rules: {
+      "@typescript-eslint/consistent-type-definitions": ["error", "type"],
+      "@typescript-eslint/explicit-function-return-type": "off",
+      "@typescript-eslint/explicit-module-boundary-types": "off",
+      "@typescript-eslint/no-explicit-any": "off",
+      "@typescript-eslint/no-unused-vars": [
+        "error",
+        {
+          argsIgnorePattern: "^_",
+          caughtErrorsIgnorePattern: "^_",
+          varsIgnorePattern: "^_",
+        },
+      ],
+      "import/order": [
+        "error",
+        {
+          alphabetize: {
+            caseInsensitive: true,
+            order: "asc",
+          },
+        },
+      ],
+    },
+  },
 ];

src/extension/.gitignore

@@ -1,3 +1,3 @@
 node_modules
 out
-web-view-dist
+view-dist

src/extension/.npmrc

@@ -1,2 +1 @@
 node-linker=hoisted
-shamefully-hoist=true

src/extension/jest.config.js renamed to src/extension/jest.config.cjs

@@ -2,15 +2,22 @@
 module.exports = {
   preset: "ts-jest",
   testEnvironment: "node",
-  roots: ["<rootDir>/src"],
-  testMatch: ["**/*.test.ts"],
+  roots: ["<rootDir>/../tests"],
+  testMatch: ["**/*.spec.ts"],
   transform: {
     "^.+\\.ts$": "ts-jest",
   },
-  collectCoverageFrom: ["src/**/*.ts", "!src/**/*.d.ts", "!src/**/*.test.ts"],
+  collectCoverageFrom: [
+    "../extension/**/*.ts",
+    "../internal/**/*.ts",
+    "../pkg/**/*.ts",
+    "!**/*.d.ts",
+    "!**/*.spec.ts",
+  ],
   coverageDirectory: "coverage",
   coverageReporters: ["text", "lcov", "html"],
   moduleFileExtensions: ["ts", "js", "json"],
+  moduleDirectories: ["node_modules", "<rootDir>/node_modules"],
   verbose: true,
   clearMocks: true,
   restoreMocks: true,