Skip to content

Conversation

@haraldschilly
Copy link
Contributor

@haraldschilly haraldschilly commented Jan 15, 2026

ref #8658

ignore *.toc

sync-doc.ts:1008 Uncaught (in promise) Error: syncstring_table must be defined
    at SyncDoc._this1.syncstring_table_get_one (sync-doc.ts:1008:1)
    at eval (sync-doc.ts:2711:1)
    at step (sync-doc.js:289:23)
    at Object.eval [as next] (sync-doc.js:230:20)
    at asyncGeneratorStep (sync-doc.js:22:28)
    at _next (sync-doc.js:40:17)
    at eval (sync-doc.js:45:13)
    at new Promise (<anonymous>)
    at eval (sync-doc.js:37:16)
    at SyncDoc._this1.handle_syncstring_update (sync-doc.ts:2726:1)

@haraldschilly haraldschilly force-pushed the fix-latex-sync-toc-8658 branch 3 times, most recently from 9331cdf to a1b3e6c Compare January 15, 2026 18:14
@haraldschilly haraldschilly marked this pull request as ready for review January 15, 2026 18:16
@haraldschilly
Copy link
Contributor Author

@codex review

@haraldschilly haraldschilly added PR-TODO-cocalc2 merge/migrate this PR into CoCalc2 in the future PR-needs review and removed PR-work in progress labels Jan 15, 2026
@chatgpt-codex-connector
Copy link

Codex Review: Didn't find any major issues. Can't wait for the next one!

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@haraldschilly haraldschilly force-pushed the fix-latex-sync-toc-8658 branch from a1b3e6c to 4815fe8 Compare January 16, 2026 10:03
@haraldschilly haraldschilly force-pushed the fix-latex-sync-toc-8658 branch from 4815fe8 to 1aaa6d2 Compare January 16, 2026 10:04
@haraldschilly
Copy link
Contributor Author

@codex review

Copy link

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 1aaa6d2374

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@haraldschilly haraldschilly merged commit 122a4a6 into master Jan 16, 2026
4 checks passed
@haraldschilly haraldschilly changed the title frontend/latex: fix syncdoc vs TOC issue and safeguard sync-doc itself frontend/latex: fix TOC issue and safeguard sync-doc itself Jan 16, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

PR-needs review PR-TODO-cocalc2 merge/migrate this PR into CoCalc2 in the future

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants