Skip to content

Fix AI features#40

Open
baxyz wants to merge 5 commits into
mainfrom
june-2-fixes
Open

Fix AI features#40
baxyz wants to merge 5 commits into
mainfrom
june-2-fixes

Conversation

@baxyz

@baxyz baxyz commented Jun 25, 2026

Copy link
Copy Markdown
Contributor

Description

Please include a summary of what this PR does and why it's needed.

Type of Change

  • Bug fix (non-breaking change that fixes an issue)
  • New feature (non-breaking change that adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Documentation update
  • Refactoring
  • Test improvement

Related Issues

Closes #(issue number)

How Has This Been Tested?

Describe the tests you ran and how to reproduce them:

  • Test A
  • Test B

Checklist

  • My code follows the code style of this project
  • I have updated the documentation accordingly
  • I have added tests for my changes
  • All new and existing tests passed locally
  • My commits follow the conventional commit format

Screenshots (if applicable)

Add screenshots for UI changes.

Additional Context

Add any other context about the PR here.

baxyz added 2 commits June 25, 2026 19:02
- include mistral-dev in PR validation jobs
- create mount sources for ~/.vibe
- update devcontainer-feature version to 1.0.1
- bump version to 1.0.3
- add command to create ~/.claude directory
@github-actions

Copy link
Copy Markdown

❌ PR Validation Failed

2/3 checks passed


📋 Pipeline Status

Job Status
🧾 Conventional Commits passing
🧪 Feature Tests failing
🐚 ShellCheck passing

🤖 Generated by @helpers4 CI • 2026-06-25

Add test/mistral-dev/test.sh to verify that the mistral-dev feature's
setup-credentials.sh script is properly installed, executable, and contains
the expected TARGET_HOME, STAGED, and symlink creation logic.

This fixes the test-features job failure that occurred after commit 215b03c
added mistral-dev to the CI test matrix without the corresponding test.

Generated by Mistral Vibe.
Co-Authored-By: Mistral Vibe <vibe@mistral.ai>
baxyz and others added 2 commits June 25, 2026 22:50
- Add fallback to version 0.9.0 when GitHub API is unavailable
- Add --max-time 10 to API curl call and use `|| true` to prevent pipeline failure
- Improve download with -sSL --max-time 30 and retry with different flags
- Verify curl is available before use with explicit install fallback
- Add git to h4_ensure_packages call for consistency

This fixes CI failures when GitHub API is rate-limited or network is restricted.

Generated by Mistral Vibe.
Co-Authored-By: Mistral Vibe <vibe@mistral.ai>
Change test command path from '.' to 'src' so devcontainer CLI can find
feature.json files in src/<feature>/ directories.

This fixes the 'Id: Unknown' error that occurred when CLI couldn't locate
feature manifests, causing git-absorb and other features to fail.

Generated by Mistral Vibe.
Co-Authored-By: Mistral Vibe <vibe@mistral.ai>
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.

1 participant