inkpen 0.7.2 → 0.8.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 5c4fef3be455d9fd017e9bdda3fb1b33ec1c1a740cc51ce3ba7dee567a6b4ba9
-  data.tar.gz: 9a7434fe2b8407f244d0de0ee1e2fecb118d51e6c06ae8d27fac08434ac6fcc0
+  metadata.gz: d494c04bfb8bd300d7530a7bf25bf25bcb9de03940d05efae4b7000056344ba4
+  data.tar.gz: 242089e92a90223d06f5ea5569df0e3cd8653c073705989b0b2c168ef8d9f65b
 SHA512:
-  metadata.gz: 5875dd8060a7d79ac6825ac7fca3f0f773226dc3dcada7d7ffb106f2ff2f963b0b8a24492d340d8467cb27cf0243f7a1f3fa2ab76185a8aa88550e29106f44ff
-  data.tar.gz: a33dc846e60d11f652173d10031f33dd0478b191384d2c18a59b1df3bf158b2dc6ec74c97d57385229e9a5307feb7bb1a6fdf0284b6e9bac0d1075270babb93a
+  metadata.gz: 8622c8eec93c586b4e588d830a8f14a1b235a38c1a88269dc4327952586e308898e4ed20b5d2fac9821af1e79c67deb5263a920969c718d3c160caacca5abf91
+  data.tar.gz: 6d4efde0df050df0601d615662443887aca6ab7f267c80ad1903cd94db63fd3c9277fe4768246db9d5419b7659c66d0a2cda6f04c5f0141e88dfd3b41bbb3240

data/CLAUDE.md

@@ -132,10 +132,70 @@ app/
 
 ## Development
 
+### Ruby side
+
 ```bash
-# Run tests
-bundle exec rake test
+bundle install
+bundle exec rake test    # Minitest unit tests
+bin/console              # IRB with Inkpen loaded
+```
 
-# Console
-bin/console
+### JavaScript side (since 0.8.0)
+
+```bash
+npm install              # installs production deps + esbuild + vitest
+npm test                 # runs the spec 01 markdown round-trip tests (vitest)
+npm run build            # rebuilds app/assets/javascripts/inkpen.bundle.js
+npm run build:check      # validates the bundle: parses, has expected
+                         # exports, contains zero third-party CDN refs
 ```
+
+**The bundle is committed to the repo.** Run `npm run build` after every
+JS change so consumers don't need to install npm.
+
+### Releasing a new version
+
+See `02-addons/107-inkpen-docs/docs/release-process.md` for the full
+release runbook. TL;DR:
+
+1. Bump `lib/inkpen/version.rb` + `inkpen.gemspec` (stub line +
+   `s.version`).
+2. `npm run build && npm run build:check && npm test &&
+   bundle exec rake test`.
+3. `bundle exec rake build` → `pkg/inkpen-X.Y.Z.gem`.
+4. Smoke test in InventList (open Nodepad, confirm zero `esm.sh`
+   requests in DevTools).
+5. Publish (path-pinned / private gemserver / rubygems — see release
+   doc).
+
+## Bundle / vendoring (since 0.8.0)
+
+`config/importmap.rb` no longer pins to `esm.sh`. All upstream
+dependencies (TipTap@2.10.3, ProseMirror@1.x, lowlight, highlight.js,
+tippy, popperjs, linkifyjs, and three third-party TipTap extensions)
+are vendored into a single ESM bundle at
+`app/assets/javascripts/inkpen.bundle.js` (1.9 MB minified, source map
+beside it).
+
+The build entrypoint is `app/assets/javascripts/inkpen/index.js`. The
+bundle preserves `import "inkpen"` as a side-effecting import that
+registers `inkpen--editor`, `inkpen--toolbar`,
+`inkpen--sticky-toolbar` Stimulus controllers on `window.Stimulus`
+(or starts its own Application if the host hasn't already).
+
+**Host externals kept out of the bundle** (host provides them):
+
+- `@hotwired/stimulus`
+- `@hotwired/turbo-rails`
+
+**Why this exists:** three production CDN incidents on 2026-05-14
+caused by importmap pinning to esm.sh. Plan and audit live at
+`02-addons/107-inkpen-docs/plans/05-vendor-bundle-cdn-decoupling.md`.
+
+## Internal planning docs
+
+Inkpen has a dedicated planning surface at
+`02-addons/107-inkpen-docs/`. Roadmap, plans, sessions, handoffs,
+audits all live there. Don't recreate that structure here; this gem
+ships its own consumer-facing docs in `docs/` (README, CHANGELOG,
+FEATURES, ROADMAP, MARKDOWN_MODE planning doc, audits).

data/app/assets/javascripts/inkpen/export/markdown.js

@@ -5,6 +5,8 @@
  * Supports frontmatter, tables, task lists, callouts, and custom blocks.
  */
 
+import { parseMarkdownToHTMLReal } from "./markdown_parser.js"
+
 // Mark serializers for inline formatting
 const MARK_SERIALIZERS = {
   bold: (text) => `**${text}**`,
@@ -74,13 +76,20 @@ export function importFromMarkdown(markdown, schema, options = {}) {
   // Parse frontmatter if present
   const { content, frontmatter } = parseFrontmatter(markdown)
 
-  // Parse markdown to HTML first (using browser or a simple parser)
-  const html = parseMarkdownToHTML(content)
+  // Parser selection. Default is the legacy regex parser to preserve
+  // behavior for existing callers. Pass `options.parser: "real"` to opt
+  // into the GFM-correct marked-based parser. Step 6 of spec 01 will flip
+  // the default once round-trip fixtures pass.
+  const parser = options.parser === "real" ? "real" : "legacy"
+  const html = parser === "real"
+    ? parseMarkdownToHTMLReal(content)
+    : parseMarkdownToHTML(content)
 
   // Return structured result
   return {
     html,
-    frontmatter
+    frontmatter,
+    parser
   }
 }