@bgx4k3p/huly-mcp-server 2.2.3 → 2.2.5

package/node_modules/prosemirror-view/src/domobserver.ts

@@ -52,16 +52,24 @@ export class DOMObserver {
     this.observer = window.MutationObserver &&
       new window.MutationObserver(mutations => {
         for (let i = 0; i < mutations.length; i++) this.queue.push(mutations[i])
-        // IE11 will sometimes (on backspacing out a single character
-        // text node after a BR node) call the observer callback
-        // before actually updating the DOM, which will cause
-        // ProseMirror to miss the change (see #930)
         if (browser.ie && browser.ie_version <= 11 && mutations.some(
           m => m.type == "childList" && m.removedNodes.length ||
-               m.type == "characterData" && m.oldValue!.length > m.target.nodeValue!.length))
+               m.type == "characterData" && m.oldValue!.length > m.target.nodeValue!.length)) {
+          // IE11 will sometimes (on backspacing out a single character
+          // text node after a BR node) call the observer callback
+          // before actually updating the DOM, which will cause
+          // ProseMirror to miss the change (see #930)
           this.flushSoon()
-        else
+        } else if (browser.safari && view.composing && mutations.some(
+          m => m.type == "childList" && m.target.nodeName == "TR")) {
+          // Safari does weird stuff when finishing a composition in a
+          // table cell, which tends to involve inserting inappropriate
+          // nodes in the table row.
+          view.input.badSafariComposition = true
+          this.flushSoon()
+        } else {
           this.flush()
+        }
       })
     if (useCharData) {
       this.onCharData = e => {
@@ -184,7 +192,20 @@ export class DOMObserver {
       }
     }
 
-    if (browser.gecko && added.length) {
+    if (added.some(n => n.nodeName == "BR") && (view.input.lastKeyCode == 8 || view.input.lastKeyCode == 46)) {
+      // Browsers sometimes insert a bogus break node if you
+      // backspace out the last bit of text before an inline-flex node (#1552)
+      for (let node of added) if (node.nodeName == "BR" && node.parentNode) {
+        let after = node.nextSibling
+        while (after && after.nodeType == 1) {
+          if ((after as HTMLElement).contentEditable == "false") {
+            node.parentNode.removeChild(node)
+            break
+          }
+          after = after.firstChild
+        }
+      }
+    } else if (browser.gecko && added.length) {
       let brs = added.filter(n => n.nodeName == "BR") as HTMLElement[]
       if (brs.length == 2) {
         let [a, b] = brs
@@ -198,15 +219,6 @@ export class DOMObserver {
             br.remove()
         }
       }
-    } else if ((browser.chrome || browser.safari) && added.some(n => n.nodeName == "BR") &&
-               (view.input.lastKeyCode == 8 || view.input.lastKeyCode == 46)) {
-      // Chrome/Safari sometimes insert a bogus break node if you
-      // backspace out the last bit of text before an inline-flex node (#1552)
-      for (let node of added) if (node.nodeName == "BR" && node.parentNode) {
-        let after = node.nextSibling
-        if (after && after.nodeType == 1 && (after as HTMLElement).contentEditable == "false")
-          node.parentNode.removeChild(node)
-      }
     }
 
     let readSel: Selection | null = null
@@ -226,6 +238,10 @@ export class DOMObserver {
         view.docView.markDirty(from, to)
         checkCSS(view)
       }
+      if (view.input.badSafariComposition) {
+        view.input.badSafariComposition = false
+        fixUpBadSafariComposition(view, added)
+      }
       this.handleDOMChange(from, to, typeOver, added)
       if (view.docView && view.docView.dirty) view.updateState(view.state)
       else if (!this.currentSelection.eq(sel)) selectionToDOM(view)
@@ -347,3 +363,30 @@ function blockParent(view: EditorView, node: DOMNode): Node | null {
   }
   return null
 }
+
+// Kludge for a Safari bug where, on ending a composition in an
+// otherwise empty table cell, it randomly moves the composed text
+// into the table row around that cell, greatly confusing everything
+// (#188).
+function fixUpBadSafariComposition(view: EditorView, addedNodes: readonly DOMNode[]) {
+  let {focusNode, focusOffset} = view.domSelectionRange()
+  for (let node of addedNodes) {
+    if (node.parentNode?.nodeName == "TR") {
+      let nextCell = node.nextSibling
+      while (nextCell && (nextCell.nodeName != "TD" && nextCell.nodeName != "TH")) nextCell = nextCell.nextSibling
+      if (nextCell) {
+        let parent = nextCell
+        for (;;) {
+          let first = parent.firstChild
+          if (!first || first.nodeType != 1 || (first as HTMLElement).contentEditable == "false" ||
+              /^(BR|IMG)$/.test(first.nodeName)) break
+          parent = first
+        }
+        parent.insertBefore(node, parent.firstChild)
+        if (focusNode == node) view.domSelection()!.collapse(node, focusOffset)
+      } else {
+        node.parentNode.removeChild(node)
+      }
+    }
+  }
+}

package/node_modules/prosemirror-view/src/index.ts

@@ -54,7 +54,7 @@ export class EditorView {
   /// @internal
   declare domObserver: DOMObserver
   /// Holds `true` when a hack node is needed in Firefox to prevent the
-  /// [space is eaten issue](https://github.com/ProseMirror/prosemirror/issues/651)
+  /// [space is eaten issue](https://code.haverbeke.berlin/prosemirror/prosemirror/issues/651)
   /// @internal
   requiresGeckoHackNode: boolean = false
 
@@ -202,7 +202,7 @@ export class EditorView {
           this.docView.destroy()
           this.docView = docViewDesc(state.doc, outerDeco, innerDeco, this.dom, this)
         }
-        if (chromeKludge && !this.trackWrites) forceSelUpdate = true
+        if (chromeKludge && (!this.trackWrites || !this.dom.contains(this.trackWrites))) forceSelUpdate = true
       }
       // Work around for an issue where an update arriving right between
       // a DOM selection change and the "selectionchange" event for it
@@ -274,11 +274,11 @@ export class EditorView {
 
   private updateDraggedNode(dragging: Dragging, prev: EditorState) {
     let sel = dragging.node!, found = -1
-    if (this.state.doc.nodeAt(sel.from) == sel.node) {
+    if (sel.from < this.state.doc.content.size && this.state.doc.nodeAt(sel.from) == sel.node) {
       found = sel.from
     } else {
       let movedPos = sel.from + (this.state.doc.content.size - prev.doc.content.size)
-      let moved = movedPos > 0 && this.state.doc.nodeAt(movedPos)
+      let moved = movedPos > 0 && movedPos < this.state.doc.content.size && this.state.doc.nodeAt(movedPos)
       if (moved == sel.node) found = movedPos
     }
     this.dragging = new Dragging(dragging.slice, dragging.move,

package/node_modules/prosemirror-view/src/input.ts

@@ -35,6 +35,7 @@ export class InputState {
   compositionNodes: ViewDesc[] = []
   compositionEndedAt = -2e8
   compositionID = 1
+  badSafariComposition = false
   // Set to a composition ID when there are pending changes at compositionend
   compositionPendingChanges = 0
   domChangeCount = 0
@@ -333,8 +334,8 @@ class MouseDown {
 
     let {selection} = view.state
     if (event.button == 0 &&
-        targetNode.type.spec.draggable && targetNode.type.spec.selectable !== false ||
-        selection instanceof NodeSelection && selection.from <= targetPos && selection.to > targetPos)
+        (targetNode.type.spec.draggable && targetNode.type.spec.selectable !== false ||
+         selection instanceof NodeSelection && selection.from <= targetPos && selection.to > targetPos))
       this.mightDrag = {
         node: targetNode,
         pos: targetPos,
@@ -504,7 +505,8 @@ editHandlers.compositionend = (view, event) => {
     view.input.compositionEndedAt = event.timeStamp
     view.input.compositionPendingChanges = view.domObserver.pendingRecords().length ? view.input.compositionID : 0
     view.input.compositionNode = null
-    if (view.input.compositionPendingChanges) Promise.resolve().then(() => view.domObserver.flush())
+    if (view.input.badSafariComposition) view.domObserver.forceFlush()
+    else if (view.input.compositionPendingChanges) Promise.resolve().then(() => view.domObserver.flush())
     view.input.compositionID++
     scheduleComposeEnd(view, 20)
   }
@@ -671,8 +673,9 @@ export class Dragging {
 const dragCopyModifier: keyof DragEvent = browser.mac ? "altKey" : "ctrlKey"
 
 function dragMoves(view: EditorView, event: DragEvent) {
-  let moves = view.someProp("dragCopies", test => !test(event))
-  return moves != null ? moves : !event[dragCopyModifier]
+  let copy: boolean | undefined
+  view.someProp("dragCopies", test => { copy = copy || test(event) })
+  return copy != null ? !copy : !event[dragCopyModifier]
 }
 
 handlers.dragstart = (view, _event) => {
@@ -699,7 +702,7 @@ handlers.dragstart = (view, _event) => {
   if (!event.dataTransfer.files.length || !browser.chrome || browser.chrome_version > 120)
     event.dataTransfer.clearData()
   event.dataTransfer.setData(brokenClipboardAPI ? "Text" : "text/html", dom.innerHTML)
-  // See https://github.com/ProseMirror/prosemirror/issues/1156
+  // See https://code.haverbeke.berlin/prosemirror/prosemirror/issues/1156
   event.dataTransfer.effectAllowed = "copyMove"
   if (!brokenClipboardAPI) event.dataTransfer.setData("text/plain", text)
   view.dragging = new Dragging(slice, dragMoves(view, event), node)

package/node_modules/prosemirror-view/src/viewdesc.ts

@@ -30,7 +30,7 @@ export type ViewMutationRecord = MutationRecord | { type: "selection", target: D
 /// Objects returned as node views must conform to this interface.
 export interface NodeView {
   /// The outer DOM node that represents the document node.
-  dom: DOMNode
+  dom: HTMLElement
 
   /// The DOM node that should hold the node's content. Only meaningful
   /// if the node view also defines a `dom` property and if its node
@@ -99,7 +99,7 @@ export interface NodeView {
 /// Objects returned as mark views must conform to this interface.
 export interface MarkView {
   /// The outer DOM node that represents the document node.
-  dom: DOMNode
+  dom: HTMLElement
 
   /// The DOM node that should hold the mark's content. When this is not
   /// present, the `dom` property is used as the content DOM.
@@ -610,7 +610,7 @@ class CompositionViewDesc extends ViewDesc {
 // some cases they will be split more often than would appear
 // necessary.
 class MarkViewDesc extends ViewDesc {
-  constructor(parent: ViewDesc, readonly mark: Mark, dom: DOMNode, contentDOM: HTMLElement, readonly spec: MarkView) {
+  constructor(parent: ViewDesc, readonly mark: Mark, dom: HTMLElement, contentDOM: HTMLElement, readonly spec: MarkView) {
     super(parent, [], dom, contentDOM)
   }
 
@@ -697,13 +697,13 @@ export class NodeViewDesc extends ViewDesc {
       if (descObj.parent) return descObj.parent.posBeforeChild(descObj)
     }, outerDeco, innerDeco)
 
-    let dom = spec && spec.dom, contentDOM = spec && spec.contentDOM
+    let dom: DOMNode | undefined = spec && spec.dom, contentDOM = spec && spec.contentDOM
     if (node.isText) {
       if (!dom) dom = document.createTextNode(node.text!)
       else if (dom.nodeType != 3) throw new RangeError("Text must be rendered as a DOM text node")
     } else if (!dom) {
       let spec = (DOMSerializer.renderSpec as any)(document, node.type.spec.toDOM!(node), null, node.attrs)
-      ;({dom, contentDOM} = spec as {dom: DOMNode, contentDOM?: HTMLElement})
+      ;({dom, contentDOM} = spec as {dom: HTMLElement, contentDOM?: HTMLElement})
     }
     if (!contentDOM && !node.isText && dom.nodeName != "BR") { // Chrome gets confused by <br contenteditable=false>
       if (!(dom as HTMLElement).hasAttribute("contenteditable")) (dom as HTMLElement).contentEditable = "false"
@@ -714,8 +714,8 @@ export class NodeViewDesc extends ViewDesc {
     dom = applyOuterDeco(dom, outerDeco, node)
 
     if (spec)
-      return descObj = new CustomNodeViewDesc(parent, node, outerDeco, innerDeco, dom, contentDOM || null, nodeDOM,
-                                              spec, view, pos + 1)
+      return descObj = new CustomNodeViewDesc(parent, node, outerDeco, innerDeco, dom as HTMLElement, contentDOM || null,
+                                              nodeDOM as HTMLElement, spec, view, pos + 1)
     else if (node.isText)
       return new TextViewDesc(parent, node, outerDeco, innerDeco, dom, nodeDOM, view)
     else
@@ -772,15 +772,15 @@ export class NodeViewDesc extends ViewDesc {
     let updater = new ViewTreeUpdater(this, localComposition && localComposition.node, view)
     iterDeco(this.node, this.innerDeco, (widget, i, insideNode) => {
       if (widget.spec.marks)
-        updater.syncToMarks(widget.spec.marks, inline, view)
+        updater.syncToMarks(widget.spec.marks, inline, view, i)
       else if ((widget.type as WidgetType).side >= 0 && !insideNode)
-        updater.syncToMarks(i == this.node.childCount ? Mark.none : this.node.child(i).marks, inline, view)
+        updater.syncToMarks(i == this.node.childCount ? Mark.none : this.node.child(i).marks, inline, view, i)
       // If the next node is a desc matching this widget, reuse it,
       // otherwise insert the widget as a new view desc.
       updater.placeWidget(widget, view, off)
     }, (child, outerDeco, innerDeco, i) => {
       // Make sure the wrapping mark descs match the node's marks.
-      updater.syncToMarks(child.marks, inline, view)
+      updater.syncToMarks(child.marks, inline, view, i)
       // Try several strategies for drawing this node
       let compIndex
       if (updater.findNodeMatch(child, outerDeco, innerDeco, i)) {
@@ -799,7 +799,7 @@ export class NodeViewDesc extends ViewDesc {
       off += child.nodeSize
     })
     // Drop all remaining descs after the current position.
-    updater.syncToMarks([], inline, view)
+    updater.syncToMarks([], inline, view, 0)
     if (this.node.isTextblock) updater.addTextblockHacks()
     updater.destroyRest()
 
@@ -985,7 +985,7 @@ class TrailingHackViewDesc extends ViewDesc {
 // customized.
 class CustomNodeViewDesc extends NodeViewDesc {
   constructor(parent: ViewDesc | undefined, node: Node, outerDeco: readonly Decoration[], innerDeco: DecorationSource,
-              dom: DOMNode, contentDOM: HTMLElement | null, nodeDOM: DOMNode, readonly spec: NodeView,
+              dom: HTMLElement, contentDOM: HTMLElement | null, nodeDOM: HTMLElement, readonly spec: NodeView,
               view: EditorView, pos: number) {
     super(parent, node, outerDeco, innerDeco, dom, contentDOM, nodeDOM, view, pos)
   }
@@ -1197,7 +1197,7 @@ class ViewTreeUpdater {
 
   // Sync the current stack of mark descs with the given array of
   // marks, reusing existing mark descs when possible.
-  syncToMarks(marks: readonly Mark[], inline: boolean, view: EditorView) {
+  syncToMarks(marks: readonly Mark[], inline: boolean, view: EditorView, parentIndex: number) {
     let keep = 0, depth = this.stack.length >> 1
     let maxKeep = Math.min(depth, marks.length)
     while (keep < maxKeep &&
@@ -1214,8 +1214,9 @@ class ViewTreeUpdater {
     }
     while (depth < marks.length) {
       this.stack.push(this.top, this.index + 1)
-      let found = -1
-      for (let i = this.index; i < Math.min(this.index + 3, this.top.children.length); i++) {
+      let found = -1, scanTo = this.top.children.length
+      if (parentIndex < this.preMatch.index) scanTo = Math.min(this.index + 3, scanTo)
+      for (let i = this.index; i < scanTo; i++) {
         let next = this.top.children[i]
         if (next.matchesMark(marks[depth]) && !this.isLocked(next.dom)) { found = i; break }
       }
@@ -1404,12 +1405,16 @@ class ViewTreeUpdater {
 
 // Iterate from the end of the fragment and array of descs to find
 // directly matching ones, in order to avoid overeagerly reusing those
-// for other nodes. Returns the fragment index of the first node that
-// is part of the sequence of matched nodes at the end of the
-// fragment.
-function preMatch(
-  frag: Fragment, parentDesc: ViewDesc
-): {index: number, matched: Map<ViewDesc, number>, matches: readonly ViewDesc[]} {
+// for other nodes.
+function preMatch(frag: Fragment, parentDesc: ViewDesc): {
+  // The fragment index of the first node that is part of the sequence
+  // of matched nodes at the end of the fragment.
+  index: number,
+  // A map from matched descs to fragment indices.
+  matched: Map<ViewDesc, number>,
+  // The matched descs.
+  matches: readonly ViewDesc[]
+} {
   let curDesc = parentDesc, descI = curDesc.children.length
   let fI = frag.childCount, matched = new Map, matches = []
   outer: while (fI > 0) {
@@ -1428,7 +1433,6 @@ function preMatch(
       } else if (curDesc == parentDesc) {
         break outer
       } else {
-        // FIXME
         descI = curDesc.parent!.children.indexOf(curDesc)
         curDesc = curDesc.parent!
       }

package/node_modules/prosemirror-view/style/prosemirror.css

@@ -23,7 +23,7 @@
 .ProseMirror-hideselection *::-moz-selection { background: transparent; }
 .ProseMirror-hideselection { caret-color: transparent; }
 
-/* See https://github.com/ProseMirror/prosemirror/issues/1421#issuecomment-1759320191 */
+/* See https://code.haverbeke.berlin/prosemirror/prosemirror/issues/1421#issuecomment-1759320191 */
 .ProseMirror [draggable][contenteditable=false] { user-select: text }
 
 .ProseMirror-selectednode {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bgx4k3p/huly-mcp-server",
-  "version": "2.2.3",
+  "version": "2.2.5",
   "description": "MCP server for Huly issue tracking with stdio and Streamable HTTP transports",
   "type": "module",
   "license": "MIT",
@@ -16,6 +16,7 @@
     "model-context-protocol",
     "issue-tracker",
     "rest-api",
+    "codex",
     "claude",
     "ai-agent"
   ],
@@ -44,8 +45,8 @@
     "version:sync": "node -e \"const{readFileSync:r,writeFileSync:w}=require('fs');const v=r('VERSION','utf-8').trim();const f='package.json';const j=JSON.parse(r(f,'utf-8'));j.version=v;w(f,JSON.stringify(j,null,2)+'\\n');console.log('Synced version '+v+' to package.json')\""
   },
   "dependencies": {
-    "@modelcontextprotocol/sdk": "^1.27.1",
-    "jsdom": "^29.0.0"
+    "@modelcontextprotocol/sdk": "^1.29.0",
+    "jsdom": "^29.1.1"
   },
   "bundleDependencies": [
     "@hcengineering/account-client",
@@ -70,6 +71,6 @@
     "@hcengineering/tracker"
   ],
   "devDependencies": {
-    "eslint": "^10.1.0"
+    "eslint": "^10.4.0"
   }
 }

package/src/client.mjs

@@ -2378,7 +2378,7 @@ export class HulyClient {
       issue._id,
       tracker.class.Issue,
       'comments',
-      { message: toMarkup(text, format), attachments: 0 },
+      { message: toCollaboratorMarkup(text, format), attachments: 0 },
       commentId
     );
 
@@ -3597,7 +3597,7 @@ export class HulyClient {
     if (!comment) throw new Error(`Comment not found: ${commentId}`);
 
     await client.updateDoc(chunter.class.ChatMessage, project._id, commentId, {
-      message: toMarkup(text, format)
+      message: toCollaboratorMarkup(text, format)
     });
 
     return {

package/src/config.mjs

@@ -8,6 +8,7 @@ export const HULY_TOKEN = process.env.HULY_TOKEN;
 export const HULY_EMAIL = process.env.HULY_EMAIL;
 export const HULY_PASSWORD = process.env.HULY_PASSWORD;
 export const HULY_WORKSPACE = process.env.HULY_WORKSPACE;
+export const HULY_PROJECT = process.env.HULY_PROJECT;
 export const HULY_CREDS = HULY_TOKEN
   ? { token: HULY_TOKEN }
   : { email: HULY_EMAIL, password: HULY_PASSWORD };

package/src/helpers.mjs

@@ -232,8 +232,8 @@ export function fromCollaboratorMarkup(markup, format = 'markdown') {
 }
 
 /**
- * Convert text to MarkupContent for non-collaborator fields
- * (milestones, comments, components, projects).
+ * Convert text to MarkupContent for SDK fields that expect MarkupContent
+ * instead of serialized ProseMirror markup strings.
  */
 export function toMarkup(text, format = 'markdown') {
   if (!text) return new MarkupContent('');

package/src/index.mjs

@@ -6,11 +6,41 @@
  *   node src/index.mjs                          → Start MCP stdio server
  *   node src/index.mjs --get-token              → Print JWT (uses HULY_EMAIL/HULY_PASSWORD env vars)
  *   node src/index.mjs --get-token -e EMAIL -p PASS -u URL
+ *   node src/index.mjs --init-codex             → Generate project .codex/config.toml
+ *   node src/index.mjs --init-claude            → Generate project .mcp.json
+ *   node src/index.mjs --init-all               → Generate Claude and Codex project config
  */
 
 const args = process.argv.slice(2);
 
-if (args.includes('--get-token')) {
+if (args.includes('--init-all')) {
+  const { initAllConfigs } = await import('./initCodex.mjs');
+  try {
+    const result = initAllConfigs(args.filter(arg => arg !== '--init-all'));
+    console.log(result.message);
+  } catch (error) {
+    console.error(error.message);
+    process.exit(1);
+  }
+} else if (args.includes('--init-claude')) {
+  const { initClaudeConfig } = await import('./initCodex.mjs');
+  try {
+    const result = initClaudeConfig(args.filter(arg => arg !== '--init-claude'));
+    console.log(result.message);
+  } catch (error) {
+    console.error(error.message);
+    process.exit(1);
+  }
+} else if (args.includes('--init-codex')) {
+  const { initCodexConfig } = await import('./initCodex.mjs');
+  try {
+    const result = initCodexConfig(args.filter(arg => arg !== '--init-codex'));
+    console.log(result.message);
+  } catch (error) {
+    console.error(error.message);
+    process.exit(1);
+  }
+} else if (args.includes('--get-token')) {
   const flag = (name) => {
     const i = args.indexOf(name);
     return i !== -1 && i + 1 < args.length ? args[i + 1] : undefined;