hale-commenting-system 3.0.0 → 3.1.1

package/GITHUB_OAUTH_ENV_TEMPLATE.md

@@ -11,8 +11,8 @@ Create `patternfly-react-seed/.env` (this file is already in `.gitignore`):
 VITE_GITHUB_CLIENT_ID=YOUR_GITHUB_OAUTH_CLIENT_ID
 
 # Target repo for Issues/Comments
-VITE_GITHUB_OWNER=JustinXHale
-VITE_GITHUB_REPO=pfseed-commenting-system
+VITE_GITHUB_OWNER=YOUR_GITHUB_USERNAME
+VITE_GITHUB_REPO=YOUR_REPO_NAME
 ```
 
 ### Server-only secret (local dev)

package/README.md

@@ -12,6 +12,15 @@ A commenting system for PatternFly React applications that allows designers and
 - **Responsive** - Works on desktop and mobile devices
 - **Easy Integration** - Automated setup script for seamless installation
 
+## Prerequisites
+
+This package was developed for **PatternFly React Seed** projects, but can be used with similar PatternFly React applications that have:
+- Webpack-based setup with `webpack.dev.js`
+- `src/app/` directory structure
+- PatternFly React Core and Icons dependencies
+
+The automated integration script (`npx hale-commenting-system init`) works best with PatternFly React Seed or projects with a similar structure.
+
 ## Installation
 
 ```bash
@@ -131,4 +140,4 @@ MIT
 
 ## Support
 
-For issues, questions, or contributions, please visit the [repository](https://github.com/patternfly/patternfly-react-seed).
+For issues or questions, please open an issue on npm or contact the package maintainer.

package/package.json

@@ -1,15 +1,21 @@
 {
   "name": "hale-commenting-system",
-  "version": "3.0.0",
+  "version": "3.1.1",
   "description": "A commenting system for PatternFly React applications that allows designers and developers to add comments directly on design pages, sync with GitHub Issues, and link Jira tickets.",
-  "repository": "https://github.com/patternfly/patternfly-react-seed.git",
   "homepage": "https://www.npmjs.com/package/hale-commenting-system",
   "license": "MIT",
   "main": "src/app/commenting-system/index.ts",
   "types": "src/app/commenting-system/index.ts",
   "bin": {
-    "hale-commenting-system": "./scripts/integrate.js"
+    "hale-commenting-system": "scripts/integrate.js"
   },
+  "files": [
+    "src/app/commenting-system/",
+    "scripts/",
+    "README.md",
+    "LICENSE",
+    "GITHUB_OAUTH_ENV_TEMPLATE.md"
+  ],
   "scripts": {
     "prebuild": "npm run type-check && npm run clean",
     "build": "webpack --config webpack.prod.js",

package/scripts/integrate.js

@@ -1359,13 +1359,43 @@ async function main() {
       }
     ]);
 
+    // Ask for target repo to store comments/issues
+    console.log('\nWhere do you want to store comments as GitHub Issues?');
+    console.log('This should be a repository you have write access to.\n');
+
+    const repoAnswers = await prompt([
+      {
+        type: 'input',
+        name: 'owner',
+        message: 'GitHub repository owner (username or organization):',
+        default: owner,
+        validate: (input) => {
+          if (!input.trim()) return 'Owner is required';
+          return true;
+        }
+      },
+      {
+        type: 'input',
+        name: 'repo',
+        message: 'GitHub repository name:',
+        default: repo,
+        validate: (input) => {
+          if (!input.trim()) return 'Repository name is required';
+          return true;
+        }
+      }
+    ]);
+
+    const targetOwner = repoAnswers.owner;
+    const targetRepo = repoAnswers.repo;
+
     // Validate GitHub credentials
     console.log('\n🔍 Validating GitHub credentials...');
     githubValid = await validateGitHubCredentials(
       githubAnswers.clientId,
       githubAnswers.clientSecret,
-      owner,
-      repo
+      targetOwner,
+      targetRepo
     );
 
     if (!githubValid) {
@@ -1378,8 +1408,8 @@ async function main() {
     githubConfig = {
       clientId: githubAnswers.clientId,
       clientSecret: githubAnswers.clientSecret,
-      owner: owner,
-      repo: repo
+      owner: targetOwner,
+      repo: targetRepo
     };
   } else {
     console.log('\n⏭️  Skipping GitHub setup. You can add it later by editing .env and .env.server files.\n');
@@ -1469,8 +1499,8 @@ async function main() {
   generateFiles({
     github: githubConfig,
     jira: jiraConfig,
-    owner: owner,
-    repo: repo
+    owner: githubConfig ? githubConfig.owner : owner,
+    repo: githubConfig ? githubConfig.repo : repo
   });
 
   // Step 5: Integrate into project

package/.claude/settings.local.json

@@ -1,7 +0,0 @@
-{
-  "permissions": {
-    "allow": [
-      "Bash(npm run type-check:*)"
-    ]
-  }
-}

package/.editorconfig

@@ -1,17 +0,0 @@
-# Editor configuration, see http://editorconfig.org
-root = true
-
-[*]
-charset = utf-8
-indent_style = space
-indent_size = 2
-insert_final_newline = true
-trim_trailing_whitespace = true
-
-[*.snap]
-max_line_length = off
-trim_trailing_whitespace = false
-
-[*.md]
-max_line_length = off
-trim_trailing_whitespace = false

package/.eslintrc.js

@@ -1,75 +0,0 @@
-module.exports = {
-  // tells eslint to use the TypeScript parser
-  "parser": "@typescript-eslint/parser",
-  // tell the TypeScript parser that we want to use JSX syntax
-  "parserOptions": {
-    "tsx": true,
-    "jsx": true,
-    "js": true,
-    "useJSXTextNode": true,
-    "project": "./tsconfig.json",
-    "tsconfigRootDir": "."
-  },
-  // we want to use the recommended rules provided from the typescript plugin
-  "extends": [
-    "eslint:recommended",
-    "plugin:react/recommended",
-    "plugin:@typescript-eslint/recommended"
-  ],
-  "globals": {
-    "window": "readonly",
-    "describe": "readonly",
-    "test": "readonly",
-    "expect": "readonly",
-    "it": "readonly",
-    "process": "readonly",
-    "document": "readonly",
-    "insights": "readonly",
-    "shallow": "readonly",
-    "render": "readonly",
-    "mount": "readonly"
-  },
-  "overrides": [
-    {
-      "files": ["src/**/*.ts", "src/**/*.tsx"],
-      "parser": "@typescript-eslint/parser",
-      "plugins": ["@typescript-eslint"],
-      "extends": ["plugin:@typescript-eslint/recommended"],
-      "rules": {
-        "react/prop-types": "off",
-        "@typescript-eslint/no-unused-vars": "error"
-      },
-    },
-  ],
-  "settings": {
-    "react": {
-      "version": "^16.11.0"
-    }
-  },
-  // includes the typescript specific rules found here: https://github.com/typescript-eslint/typescript-eslint/tree/master/packages/eslint-plugin#supported-rules
-  "plugins": [
-    "@typescript-eslint",
-    "react-hooks",
-    "eslint-plugin-react-hooks"
-  ],
-  "rules": {
-    "sort-imports": [
-      "error",
-      {
-        "ignoreDeclarationSort": true
-      }
-    ],
-    "@typescript-eslint/explicit-function-return-type": "off",
-    "react-hooks/rules-of-hooks": "error",
-    "react-hooks/exhaustive-deps": "warn",
-    "@typescript-eslint/interface-name-prefix": "off",
-    "prettier/prettier": "off",
-    "import/no-unresolved": "off",
-    "import/extensions": "off",
-    "react/prop-types": "off"
-  },
-  "env": {
-    "browser": true,
-    "node": true
-  }
-}

package/.prettierignore

@@ -1 +0,0 @@
-package.json

package/.prettierrc

@@ -1,4 +0,0 @@
-{
-  "singleQuote": true,
-  "printWidth": 120
-}

package/src/app/AppLayout/AppLayout.tsx

@@ -1,248 +0,0 @@
-import * as React from 'react';
-import { NavLink, useLocation } from 'react-router-dom';
-import {
-  Button,
-  Masthead,
-  MastheadBrand,
-  MastheadLogo,
-  MastheadMain,
-  MastheadToggle,
-  Nav,
-  NavExpandable,
-  NavItem,
-  NavList,
-  Page,
-  PageSidebar,
-  PageSidebarBody,
-  SkipToContent,
-  Switch,
-} from '@patternfly/react-core';
-import { IAppRoute, IAppRouteGroup, routes } from '@app/routes';
-import { BarsIcon, ExternalLinkAltIcon, GithubIcon } from '@patternfly/react-icons';
-import { CommentOverlay, CommentPanel, useComments, useGitHubAuth } from '@app/commenting-system';
-
-interface IAppLayout {
-  children: React.ReactNode;
-}
-
-const AppLayout: React.FunctionComponent<IAppLayout> = ({ children }) => {
-  const [sidebarOpen, setSidebarOpen] = React.useState(true);
-  const { commentsEnabled, setCommentsEnabled, drawerPinnedOpen, setDrawerPinnedOpen, floatingWidgetMode, setFloatingWidgetMode } = useComments();
-  const { isAuthenticated, user, login, logout } = useGitHubAuth();
-  const masthead = (
-    <Masthead>
-      <MastheadMain>
-        <MastheadToggle>
-          <Button
-            icon={<BarsIcon />}
-            variant="plain"
-            onClick={() => setSidebarOpen(!sidebarOpen)}
-            aria-label="Global navigation"
-          />
-        </MastheadToggle>
-        <MastheadBrand data-codemods>
-          <MastheadLogo data-codemods>
-            <svg height="40px" viewBox="0 0 679 158">
-              <title>PatternFly logo</title>
-              <defs>
-                <linearGradient x1="68%" y1="2.25860997e-13%" x2="32%" y2="100%" id="linearGradient-basic-masthead">
-                  <stop stopColor="#2B9AF3" offset="0%"></stop>
-                  <stop stopColor="#73BCF7" stopOpacity="0.502212631" offset="100%"></stop>
-                </linearGradient>
-              </defs>
-              <g stroke="none" strokeWidth="1" fill="none" fillRule="evenodd">
-                <g
-                  transform="translate(206.000000, 45.750000)"
-                  fill="var(--pf-t--global--text--color--regular)"
-                  fillRule="nonzero"
-                >
-                  <path d="M0,65.25 L0,2.25 L33.21,2.25 C37.35,2.25 41.025,3.135 44.235,4.905 C47.445,6.675 49.98,9.09 51.84,12.15 C53.7,15.21 54.63,18.72 54.63,22.68 C54.63,26.46 53.7,29.865 51.84,32.895 C49.98,35.925 47.43,38.31 44.19,40.05 C40.95,41.79 37.29,42.66 33.21,42.66 L15.48,42.66 L15.48,65.25 L0,65.25 Z M15.48,29.88 L31.41,29.88 C33.69,29.88 35.52,29.22 36.9,27.9 C38.28,26.58 38.97,24.87 38.97,22.77 C38.97,20.61 38.28,18.855 36.9,17.505 C35.52,16.155 33.69,15.48 31.41,15.48 L15.48,15.48 L15.48,29.88 Z"></path>
-                  <path d="M77.04,66.06 C73.68,66.06 70.695,65.43 68.085,64.17 C65.475,62.91 63.435,61.17 61.965,58.95 C60.495,56.73 59.76,54.18 59.76,51.3 C59.76,46.74 61.485,43.215 64.935,40.725 C68.385,38.235 73.2,36.99 79.38,36.99 C83.1,36.99 86.7,37.44 90.18,38.34 L90.18,36 C90.18,31.26 87.15,28.89 81.09,28.89 C77.49,28.89 72.69,30.15 66.69,32.67 L61.47,21.96 C69.15,18.48 76.56,16.74 83.7,16.74 C90.3,16.74 95.43,18.315 99.09,21.465 C102.75,24.615 104.58,29.04 104.58,34.74 L104.58,65.25 L90.18,65.25 L90.18,62.37 C88.26,63.69 86.235,64.635 84.105,65.205 C81.975,65.775 79.62,66.06 77.04,66.06 Z M73.62,51.03 C73.62,52.53 74.28,53.7 75.6,54.54 C76.92,55.38 78.75,55.8 81.09,55.8 C84.69,55.8 87.72,55.05 90.18,53.55 L90.18,47.43 C87.42,46.71 84.54,46.35 81.54,46.35 C79.02,46.35 77.07,46.755 75.69,47.565 C74.31,48.375 73.62,49.53 73.62,51.03 Z"></path>
-                  <path d="M137.25,65.88 C125.73,65.88 119.97,60.84 119.97,50.76 L119.97,29.79 L110.34,29.79 L110.34,17.64 L119.97,17.64 L119.97,5.4 L134.55,2.25 L134.55,17.64 L147.87,17.64 L147.87,29.79 L134.55,29.79 L134.55,47.88 C134.55,49.98 135.015,51.465 135.945,52.335 C136.875,53.205 138.51,53.64 140.85,53.64 C143.01,53.64 145.2,53.31 147.42,52.65 L147.42,64.44 C146.1,64.86 144.42,65.205 142.38,65.475 C140.34,65.745 138.63,65.88 137.25,65.88 Z"></path>
-                  <path d="M177.57,65.88 C166.05,65.88 160.29,60.84 160.29,50.76 L160.29,29.79 L150.66,29.79 L150.66,17.64 L160.29,17.64 L160.29,5.4 L174.87,2.25 L174.87,17.64 L188.19,17.64 L188.19,29.79 L174.87,29.79 L174.87,47.88 C174.87,49.98 175.335,51.465 176.265,52.335 C177.195,53.205 178.83,53.64 181.17,53.64 C183.33,53.64 185.52,53.31 187.74,52.65 L187.74,64.44 C186.42,64.86 184.74,65.205 182.7,65.475 C180.66,65.745 178.95,65.88 177.57,65.88 Z"></path>
-                  <path d="M217.62,66.15 C212.76,66.15 208.365,65.055 204.435,62.865 C200.505,60.675 197.4,57.72 195.12,54 C192.84,50.28 191.7,46.11 191.7,41.49 C191.7,36.87 192.795,32.7 194.985,28.98 C197.175,25.26 200.16,22.305 203.94,20.115 C207.72,17.925 211.92,16.83 216.54,16.83 C221.22,16.83 225.36,17.955 228.96,20.205 C232.56,22.455 235.395,25.53 237.465,29.43 C239.535,33.33 240.57,37.8 240.57,42.84 L240.57,46.44 L206.64,46.44 C207.6,48.66 209.1,50.475 211.14,51.885 C213.18,53.295 215.58,54 218.34,54 C222.42,54 225.6,52.8 227.88,50.4 L237.51,58.95 C234.51,61.47 231.435,63.3 228.285,64.44 C225.135,65.58 221.58,66.15 217.62,66.15 Z M206.37,36.27 L226.26,36.27 C225.48,33.99 224.205,32.16 222.435,30.78 C220.665,29.4 218.61,28.71 216.27,28.71 C213.87,28.71 211.8,29.37 210.06,30.69 C208.32,32.01 207.09,33.87 206.37,36.27 Z"></path>
-                  <path d="M247.41,65.25 L247.41,17.64 L261.99,17.64 L261.99,22.41 C265.23,18.51 269.4,16.56 274.5,16.56 C277.08,16.62 278.91,17.01 279.99,17.73 L279.99,30.42 C277.95,29.46 275.64,28.98 273.06,28.98 C270.78,28.98 268.665,29.505 266.715,30.555 C264.765,31.605 263.19,33.09 261.99,35.01 L261.99,65.25 L247.41,65.25 Z"></path>
-                  <path d="M286.29,65.25 L286.29,17.64 L300.87,17.64 L300.87,20.88 C304.47,18.12 308.73,16.74 313.65,16.74 C317.37,16.74 320.655,17.55 323.505,19.17 C326.355,20.79 328.59,23.04 330.21,25.92 C331.83,28.8 332.64,32.13 332.64,35.91 L332.64,65.25 L318.06,65.25 L318.06,37.89 C318.06,35.25 317.28,33.15 315.72,31.59 C314.16,30.03 312.06,29.25 309.42,29.25 C305.76,29.25 302.91,30.51 300.87,33.03 L300.87,65.25 L286.29,65.25 Z"></path>
-                  <polygon points="342 65.25 342 2.25 392.04 2.25 392.04 15.66 357.48 15.66 357.48 27.45 380.52 27.45 380.52 40.41 357.48 40.41 357.48 65.25"></polygon>
-                  <polygon points="399.96 65.25 399.96 2.25 414.54 0 414.54 65.25"></polygon>
-                  <path d="M429.21,84.69 C428.07,84.69 426.96,84.645 425.88,84.555 C424.8,84.465 423.9,84.33 423.18,84.15 L423.18,71.73 C424.38,71.97 425.88,72.09 427.68,72.09 C432.36,72.09 435.51,70.05 437.13,65.97 L437.13,65.88 L418.86,17.64 L434.97,17.64 L445.5,47.61 L457.74,17.64 L473.49,17.64 L452.16,67.68 C450.42,71.82 448.5,75.135 446.4,77.625 C444.3,80.115 441.87,81.915 439.11,83.025 C436.35,84.135 433.05,84.69 429.21,84.69 Z"></path>
-                </g>
-                <g transform="translate(0.000000, 0.000000)">
-                  <path
-                    d="M61.826087,0 L158,0 L158,96.173913 L147.695652,96.173913 C100.271201,96.173913 61.826087,57.7287992 61.826087,10.3043478 L61.826087,0 L61.826087,0 Z"
-                    fill="#0066CC"
-                  ></path>
-                  <path
-                    d="M158,3.43478261 L65.2608696,158 L138,158 C149.045695,158 158,149.045695 158,138 L158,3.43478261 L158,3.43478261 Z"
-                    fill="url(#linearGradient-basic-masthead)"
-                  ></path>
-                  <path
-                    d="M123.652174,-30.9130435 L30.9130435,123.652174 L103.652174,123.652174 C114.697869,123.652174 123.652174,114.697869 123.652174,103.652174 L123.652174,-30.9130435 L123.652174,-30.9130435 Z"
-                    fill="url(#linearGradient-basic-masthead)"
-                    transform="translate(77.282609, 46.369565) scale(1, -1) rotate(90.000000) translate(-77.282609, -46.369565) "
-                  ></path>
-                </g>
-              </g>
-            </svg>
-          </MastheadLogo>
-        </MastheadBrand>
-      </MastheadMain>
-    </Masthead>
-  );
-
-  const location = useLocation();
-
-  const renderNavItem = (route: IAppRoute, index: number) => (
-    <NavItem key={`${route.label}-${index}`} id={`${route.label}-${index}`} isActive={route.path === location.pathname}>
-      <NavLink
-        to={route.path}
-      >
-        {route.label}
-      </NavLink>
-    </NavItem>
-  );
-
-  const renderNavGroup = (group: IAppRouteGroup, groupIndex: number) => {
-    // Special handling for Comments group
-    if (group.label === 'Comments') {
-      return (
-        <NavExpandable
-          key={`${group.label}-${groupIndex}`}
-          id={`${group.label}-${groupIndex}`}
-          title="Hale Commenting System"
-          isActive={group.routes.some((route) => route.path === location.pathname)}
-        >
-          <NavItem
-            onClick={(e) => {
-              e.stopPropagation();
-              setFloatingWidgetMode(!floatingWidgetMode);
-              if (!floatingWidgetMode) {
-                setDrawerPinnedOpen(false); // Close drawer if opening floating widget
-              }
-            }}
-            style={{ cursor: 'pointer' }}
-          >
-            <div style={{ display: 'flex', alignItems: 'center', gap: '0.5rem' }}>
-              <ExternalLinkAltIcon />
-              <span>{floatingWidgetMode ? 'Close widget' : 'Pop out'}</span>
-            </div>
-          </NavItem>
-          <NavItem>
-            <div
-              data-comment-controls
-              style={{ display: 'flex', alignItems: 'center', justifyContent: 'space-between', paddingRight: '1rem' }}
-            >
-              <span>Enable Comments</span>
-              <Switch
-                id="comments-enabled-switch"
-                isChecked={commentsEnabled}
-                onChange={(_event, checked) => {
-                  setCommentsEnabled(checked);
-                  if (checked) {
-                    setDrawerPinnedOpen(true);
-                  }
-                }}
-                aria-label="Enable or disable comments"
-              />
-            </div>
-          </NavItem>
-          <NavItem>
-            <div
-              data-comment-controls
-              style={{ display: 'flex', alignItems: 'center', justifyContent: 'space-between', paddingRight: '1rem' }}
-            >
-              <span>Page info drawer</span>
-              <Switch
-                id="page-info-drawer-switch"
-                isChecked={drawerPinnedOpen}
-                onChange={(_event, checked) => setDrawerPinnedOpen(checked)}
-                aria-label="Pin page info drawer open"
-              />
-            </div>
-          </NavItem>
-          <NavItem>
-            <div
-              data-comment-controls
-              style={{ display: 'flex', alignItems: 'center', justifyContent: 'space-between', paddingRight: '1rem' }}
-            >
-              {isAuthenticated ? (
-                <div style={{ display: 'flex', alignItems: 'center', gap: '8px' }}>
-                  <span style={{ display: 'inline-flex', alignItems: 'center', gap: '6px' }}>
-                    <GithubIcon />
-                    {user?.login ? `@${user.login}` : 'Signed in'}
-                  </span>
-                  <Button variant="link" isInline onClick={logout}>
-                    Sign out
-                  </Button>
-                </div>
-              ) : (
-                <Button variant="link" isInline icon={<GithubIcon />} onClick={login}>
-                  Sign in with GitHub
-                </Button>
-              )}
-            </div>
-          </NavItem>
-          {group.routes.map((route, idx) => route.label && renderNavItem(route, idx))}
-        </NavExpandable>
-      );
-    }
-
-    // Default handling for other groups
-    return (
-      <NavExpandable
-        key={`${group.label}-${groupIndex}`}
-        id={`${group.label}-${groupIndex}`}
-        title={group.label}
-        isActive={group.routes.some((route) => route.path === location.pathname)}
-      >
-        {group.routes.map((route, idx) => route.label && renderNavItem(route, idx))}
-      </NavExpandable>
-    );
-  };
-
-  const Navigation = (
-    <Nav id="nav-primary-simple">
-      <NavList id="nav-list-simple">
-        {routes.map(
-          (route, idx) => route.label && (!route.routes ? renderNavItem(route, idx) : renderNavGroup(route, idx)),
-        )}
-      </NavList>
-    </Nav>
-  );
-
-  const Sidebar = (
-    <PageSidebar>
-      <PageSidebarBody>{Navigation}</PageSidebarBody>
-    </PageSidebar>
-  );
-
-  const pageId = 'primary-app-container';
-
-  const PageSkipToContent = (
-    <SkipToContent
-      onClick={(event) => {
-        event.preventDefault();
-        const primaryContentContainer = document.getElementById(pageId);
-        primaryContentContainer?.focus();
-      }}
-      href={`#${pageId}`}
-    >
-      Skip to Content
-    </SkipToContent>
-  );
-  return (
-    <Page
-      mainContainerId={pageId}
-      masthead={masthead}
-      sidebar={sidebarOpen && Sidebar}
-      skipToContent={PageSkipToContent}
-    >
-      <CommentPanel>
-        <CommentOverlay />
-        {children}
-      </CommentPanel>
-    </Page>
-  );
-};
-
-export { AppLayout };