Merge pull request #1 from dlabaj/refactor

fix: Cleaned up and refactor code.

Author: dlabaj

.eslintrc.js

@@ -29,7 +29,7 @@ module.exports = {
   },
   "overrides": [
     {
-      "files": ["src/**/*.ts", "src/**/*.tsx"],
+      "files": ["src/**/*.ts", "src/**/*.tsx", "demo/**/*.ts", "demo/**/*.tsx"],
       "parser": "@typescript-eslint/parser",
       "plugins": ["@typescript-eslint"],
       "extends": ["plugin:@typescript-eslint/recommended"],

.gitignore

@@ -1,5 +1,6 @@
 **/node_modules
 dist
+demo/dist
 yarn-error.log
 yarn.lock
 stats.json

.npmignore

@@ -1,47 +1,36 @@
 # Dependencies
 node_modules/
 
-# Development files  
+# Demo app (not published)
+demo/
+
+# Development files
 *.md
 !README.md
 !LICENSE
 !GITHUB_OAUTH_ENV_TEMPLATE.md
 
-# Build files
+# Build artifacts
 dist/
+demo/dist/
 build/
 stats.json
+coverage/
 
 # Development configs
 .editorconfig
 .eslintrc.js
 .prettierignore
 .prettierrc
-webpack.dev.js
-webpack.common.js
 tsconfig.json
 vitest.config.ts
-stylePaths.js
-webpack.prod.js
-
-# Source files not needed in package
-src/app/Dashboard/
-src/app/NotFound/
-src/app/Settings/
-src/app/Support/
-src/app/app.css
-src/app/app.test.tsx
-src/app/bgimages/
-src/app/utils/
-src/favicon.png
-src/index.tsx
-src/typings.d.ts
 
 # Tests
 **/*.test.ts
 **/*.test.tsx
 **/*.spec.ts
 **/*.spec.tsx
+**/__snapshots__/
 
 # Git
 .git/

README.md

@@ -207,10 +207,17 @@ npm run start:dev
 
 ## Development
 
+This repository contains two parts:
+
+- **`src/commenting-system/`** — the npm package (published to npm)
+- **`demo/`** — a PatternFly demo app for local testing (not published)
+
 ```bash
 npm install          # Install dependencies
-npm run start:dev    # Start dev server
-npm run build        # Production build
+npm run start:dev    # Start demo dev server (http://localhost:9000)
+npm run build        # Build demo app to demo/dist
+npm run type-check   # Type-check the package
+npm run type-check:demo  # Type-check the demo app
 ```
 
 ## License

src/app/AppLayout/AppLayout.tsx demo/src/app/AppLayout/AppLayout.tsxsrc/app/AppLayout/AppLayout.tsx renamed to demo/src/app/AppLayout/AppLayout.tsx

@@ -19,7 +19,7 @@ import {
 } from '@patternfly/react-core';
 import { IAppRoute, IAppRouteGroup, routes } from '@app/routes';
 import { BarsIcon, ExternalLinkAltIcon, GithubIcon } from '@patternfly/react-icons';
-import { CommentPanel, useComments, useGitHubAuth } from '@app/commenting-system';
+import { CommentPanel, useComments, useGitHubAuth } from '@design-comments';
 
 interface IAppLayout {
   children: React.ReactNode;

src/app/Comments/Comments.tsx demo/src/app/Comments/Comments.tsxsrc/app/Comments/Comments.tsx renamed to demo/src/app/Comments/Comments.tsx

@@ -12,7 +12,7 @@ import {
   Title,
 } from '@patternfly/react-core';
 import { AngleDownIcon, AngleRightIcon, SearchIcon } from '@patternfly/react-icons';
-import { useComments } from '@app/commenting-system';
+import { useComments } from '@design-comments';
 
 const Comments: React.FunctionComponent = () => {
   const navigate = useNavigate();