Skip to content

chore: initial public snapshot after M15 local-only descope#1

Merged
yono-code merged 2 commits into
mainfrom
seed/m15-public-initial
Mar 2, 2026
Merged

chore: initial public snapshot after M15 local-only descope#1
yono-code merged 2 commits into
mainfrom
seed/m15-public-initial

Conversation

@yono-code
Copy link
Copy Markdown
Owner

Summary

  • imports latest Project-ARIA snapshot after M15 completion
  • removes network feature flag path from shipped code
  • adds no-network guard and CI/PR checks
  • keeps public scope limited via export rules

Verification

  • npm run no-network:guard
  • npm run build:all
  • npm run test

@yono-code yono-code merged commit 43e5dcd into main Mar 2, 2026
1 check failed
@yono-code yono-code deleted the seed/m15-public-initial branch March 2, 2026 13:19
Copy link
Copy Markdown

@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: 881c96e6e1

ℹ️ 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".

Comment thread src/extension.ts
try {
const now = new Date().toISOString().replace(/[:.]/g, '-');
const extension = request.format === 'markdown' ? 'md' : request.format;
const defaultUri = vscode.Uri.file(`${workspacePath}\\mindmap-export-${now}.${extension}`);
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Use path.join when building mindmap export default URI

The export dialog default path is constructed with a hardcoded Windows separator (\), so on macOS/Linux the filename contains a literal backslash instead of a directory boundary. In those environments, the save dialog opens with an incorrect default target/name for every mindmap export, which is a cross-platform regression for a core user flow.

Useful? React with 👍 / 👎.

children.forEach((childId) => visit(childId, depth + 1));
};

roots.forEach((root) => visit(root.id, 0));
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Handle no-root graphs in Markdown mindmap export

When users create a cyclic mindmap (all nodes have incoming edges), roots becomes empty; the visibility helper already has a fallback for this case, but Markdown export still traverses only roots, so it emits no node list even though nodes are present. This produces empty/near-empty Markdown exports for valid editor states reachable via edge connections.

Useful? React with 👍 / 👎.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants