Skip to content

fix: minor fixes#2785

Merged
davydkov merged 1 commit intomainfrom
minor-fixes
Mar 20, 2026
Merged

fix: minor fixes#2785
davydkov merged 1 commit intomainfrom
minor-fixes

Conversation

@davydkov
Copy link
Copy Markdown
Member

@davydkov davydkov commented Mar 20, 2026 •
edited by coderabbitai bot
Loading

Summary

  • Fix logger initialization defaults and environment detection
  • Fix drag behavior for group nodes
  • Restore Markdown syntax highlighting in triple-quoted strings

🤖 Generated with Claude Code

Summary by CodeRabbit

  • Bug Fixes
    • Fixed drag behavior when moving group nodes in diagrams
    • Corrected logger initialization to properly detect development and CI environments
    • Restored Markdown syntax highlighting in the VS Code extension for better code readability

@davydkov @claude
fix: minor fixes (logger init, group node drag, markdown highlighting)
cc5e70a 
- Fix logger initialization defaults and environment detection
- Fix drag behavior for group nodes
- Restore Markdown syntax highlighting in triple-quoted strings

Co-Authored-By: Claude Opus 4.6 (1M context) <[email protected]>
@changeset-bot
Copy link
Copy Markdown

changeset-bot bot commented Mar 20, 2026

🦋 Changeset detected

Latest commit: cc5e70a

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 20 packages
Name Type
@likec4/diagram Patch
likec4 Patch
@likec4/language-services Patch
likec4-vscode Patch
@likec4/playground Patch
@likec4/react Patch
@likec4/vscode-preview Patch
@likec4/docs-astro Patch
@likec4/mcp Patch
@likec4/vite-plugin Patch
@likec4/style-preset Patch
@likec4/styles Patch
@likec4/config Patch
@likec4/core Patch
@likec4/generators Patch
@likec4/language-server Patch
@likec4/layouts Patch
@likec4/leanix-bridge Patch
@likec4/log Patch
@likec4/tsconfig Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@davydkov davydkov merged commit eddfe46 into main Mar 20, 2026
15 of 16 checks passed
@davydkov davydkov deleted the minor-fixes branch March 20, 2026 10:41
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Mar 20, 2026

Caution

Review failed

Pull request was closed or merged during review

📝 Walkthrough

Walkthrough

Multiple patches addressing diagram drag behavior, logger initialization defaults, environment detection logic migration from esm-env to std-env, default options consolidation in language services, and Markdown syntax highlighting restoration in VSCode.

Changes

Cohort / File(s) Summary
Changeset metadata
.changeset/fix-drag-group-nodes.md, .changeset/fix-logger-initialization.md, .changeset/markdown-syntax-highlighting.md		 Added patch-level changesets for diagram drag behavior fixes, logger initialization defaults, and Markdown syntax highlighting restoration.
Diagram node handling
packages/diagram/src/likec4diagram/xyflow-diagram/diagram-view.ts		 Removed dragHandle property from compound/group nodes to fix drag behavior.
Logger and environment defaults
packages/language-services/src/node/configureLogger.ts, packages/language-services/src/node/without-mcp/index.ts, packages/likec4/src/cli/index.ts		 Updated logger defaults from 'console' to false, integrated DefaultInitOptions into initialization, and replaced esm-env with std-env for environment-aware logging and update notification behavior.
VSCode syntax highlighting
packages/vscode/likec4.tmLanguage.json		 Added markdown-string pattern to enable Markdown syntax highlighting within triple-quoted strings.
Code cleanup
packages/likec4/src/cli/serve/serve.ts		 Removed commented-out logger configuration option.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

Possibly related PRs

Poem

🐰 Logger defaults rest anew,
Drag handles freed to dance true!
Environments hop with std-env grace,
Markdown glows in its rightful place,
These little patches make things right! ✨

🚥 Pre-merge checks | ❌ 3

❌ Failed checks (1 warning, 2 inconclusive)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 75.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
Title check ❓ Inconclusive The title 'fix: minor fixes' is vague and generic, using non-descriptive language that doesn't clearly convey what was actually fixed. Use a more specific title that identifies the primary fix, such as 'fix: logger initialization, drag behavior, and markdown highlighting' or focus on the most impactful change.
Description check ❓ Inconclusive The description lists the fixes but lacks required checklist sections from the template; it provides only a summary without addressing contribution guidelines, testing, or documentation status. Add the full checklist from the template with appropriate checkmarks or unchecked items to indicate which requirements were addressed or skipped.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch minor-fixes
📝 Coding Plan
  • Generate coding plan for human review comments

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@likec4-ci likec4-ci bot mentioned this pull request Mar 20, 2026
Merged
@coderabbitai coderabbitai bot mentioned this pull request Mar 20, 2026
Merged
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@davydkov