confluence-cli 1.32.0 → 1.33.0

package/README.md

@@ -249,6 +249,30 @@ Or add `"forceCloud": true` to your profile in `~/.confluence-cli/config.json`:
 }
 ```
 
+**Link rendering on Cloud (`linkStyle`):**
+
+Some Cloud instances — particularly custom-domain Cloud setups — fail to render smart links (`<a data-card-appearance="inline">`) and show "Cannot handle: DefaultLink" errors instead. If you hit this, set `linkStyle` to `plain` to emit simple `<a href>` tags, which render reliably everywhere:
+
+```bash
+export CONFLUENCE_LINK_STYLE=plain
+```
+
+Or per-profile:
+
+```json
+{
+  "profiles": {
+    "default": {
+      "domain": "wiki.example.org",
+      "forceCloud": true,
+      "linkStyle": "plain"
+    }
+  }
+}
+```
+
+Valid values: `smart` (Cloud smart links), `plain` (simple `<a href>`), `wiki` (Server/DC `ac:link`). When unset, the CLI picks `smart` for Cloud and `wiki` for Server/DC — existing behavior is unchanged.
+
 **Read-only mode** (recommended for AI agents):
 ```bash
 export CONFLUENCE_READ_ONLY=true

package/lib/config.js

@@ -17,6 +17,23 @@ const AUTH_CHOICES = [
 
 const AUTH_TYPES = ['basic', 'bearer', 'mtls', 'cookie'];
 
+const { VALID_LINK_STYLES } = require('./macro-converter');
+
+const normalizeLinkStyle = (rawValue, source) => {
+  if (rawValue === undefined || rawValue === null || rawValue === '') {
+    return undefined;
+  }
+  const value = String(rawValue).trim().toLowerCase();
+  if (VALID_LINK_STYLES.includes(value)) {
+    return value;
+  }
+  const label = source ? `${source} ` : '';
+  console.error(chalk.yellow(
+    `⚠ Invalid linkStyle ${label}"${rawValue}"; valid values: ${VALID_LINK_STYLES.join(', ')}. Falling back to auto-detection.`
+  ));
+  return undefined;
+};
+
 const isValidProfileName = (name) => /^[a-zA-Z0-9_-]+$/.test(name);
 
 const requiredInput = (label) => (input) => {
@@ -719,6 +736,7 @@ function getConfig(profileName) {
   const envProtocol = process.env.CONFLUENCE_PROTOCOL;
   const envReadOnly = process.env.CONFLUENCE_READ_ONLY;
   const envForceCloud = process.env.CONFLUENCE_FORCE_CLOUD;
+  const envLinkStyle = normalizeLinkStyle(process.env.CONFLUENCE_LINK_STYLE, 'from CONFLUENCE_LINK_STYLE');
   const envCookie = process.env.CONFLUENCE_COOKIE;
   const envMtls = normalizeMtlsConfig({
     caCert: process.env.CONFLUENCE_TLS_CA_CERT,
@@ -772,7 +790,8 @@ function getConfig(profileName) {
       authType,
       mtls: envMtls,
       readOnly: envReadOnly === 'true',
-      forceCloud: envForceCloud === 'true'
+      forceCloud: envForceCloud === 'true',
+      linkStyle: envLinkStyle
     };
   }
 
@@ -844,6 +863,9 @@ function getConfig(profileName) {
       ? envForceCloud === 'true'
       : Boolean(storedConfig.forceCloud);
 
+    const linkStyle = envLinkStyle
+      ?? normalizeLinkStyle(storedConfig.linkStyle, `in profile "${targetProfile}"`);
+
     return {
       domain: trimmedDomain,
       protocol: normalizeProtocol(storedConfig.protocol),
@@ -854,7 +876,8 @@ function getConfig(profileName) {
       authType,
       mtls,
       readOnly,
-      forceCloud
+      forceCloud,
+      linkStyle
     };
   } catch (error) {
     console.error(chalk.red('❌ Error reading configuration file:'), error.message);

package/lib/confluence-client.js

@@ -47,6 +47,7 @@ class ConfluenceClient {
       isCloud: this.isCloud(),
       webUrlPrefix: this.webUrlPrefix,
       buildUrl: (pathOrUrl) => this.buildUrl(pathOrUrl),
+      linkStyle: config.linkStyle,
     });
     this.markdown = this.converter.markdown;
 

package/lib/macro-converter.js

@@ -1,11 +1,16 @@
 const MarkdownIt = require('markdown-it');
 const { htmlToMarkdown } = require('./html-to-markdown');
 
+const VALID_LINK_STYLES = ['smart', 'plain', 'wiki'];
+
 class MacroConverter {
-  constructor({ isCloud = false, webUrlPrefix = '', buildUrl = null } = {}) {
+  constructor({ isCloud = false, webUrlPrefix = '', buildUrl = null, linkStyle = null } = {}) {
     this._isCloud = isCloud;
     this.webUrlPrefix = webUrlPrefix;
     this.buildUrl = buildUrl || ((pathOrUrl) => pathOrUrl);
+    this.linkStyle = VALID_LINK_STYLES.includes(linkStyle)
+      ? linkStyle
+      : (isCloud ? 'smart' : 'wiki');
     this.markdown = new MarkdownIt();
     this.setupConfluenceMarkdownExtensions();
   }
@@ -107,11 +112,17 @@ class MacroConverter {
     storage = storage.replace(/<th>(.*?)<\/th>/g, '<th><p>$1</p></th>');
     storage = storage.replace(/<td>(.*?)<\/td>/g, '<td><p>$1</p></td>');
 
-    if (this.isCloud()) {
+    // Convert links based on linkStyle:
+    //   "smart" — Cloud smart links (<a data-card-appearance="inline">)
+    //   "plain" — simple <a href>; workaround for "Cannot handle: DefaultLink"
+    //             errors on custom-domain Cloud instances
+    //   "wiki"  — Server/DC ac:link + ri:url storage format
+    if (this.linkStyle === 'smart') {
       storage = storage.replace(/<a href="(.*?)">(.*?)<\/a>/g, '<a href="$1" data-card-appearance="inline">$2</a>');
-    } else {
+    } else if (this.linkStyle === 'wiki') {
       storage = storage.replace(/<a href="(.*?)">(.*?)<\/a>/g, '<ac:link><ri:url ri:value="$1" /><ac:plain-text-link-body><![CDATA[$2]]></ac:plain-text-link-body></ac:link>');
     }
+    // "plain" — leave <a href> tags as-is
 
     storage = storage.replace(/<hr\s*\/?>/g, '<hr />');
 
@@ -296,3 +307,4 @@ class MacroConverter {
 }
 
 module.exports = MacroConverter;
+module.exports.VALID_LINK_STYLES = VALID_LINK_STYLES;