Skip to content

Unset CDPATH while getting SCRIPT_DIR#1039

Merged
localden merged 1 commit into
github:mainfrom
sgillis:fix-cdpath-issue
Nov 15, 2025
Merged

Unset CDPATH while getting SCRIPT_DIR#1039
localden merged 1 commit into
github:mainfrom
sgillis:fix-cdpath-issue

Conversation

@sgillis

@sgillis sgillis commented Oct 25, 2025

Copy link
Copy Markdown

Fixes #1038

Claude Code was used in debugging the issue, manually checked that this fixed the reported issue.

@sgillis sgillis requested a review from localden as a code owner October 25, 2025 09:37
@codedmart

Copy link
Copy Markdown

This resolve my issues I was seeing and posted here: #973.

@localden localden merged commit f205fa3 into github:main Nov 15, 2025
nguyenvanduocit added a commit to nguyenvanduocit/research-kit that referenced this pull request Nov 22, 2025
Synced improvements from upstream spec-kit:

**Bug Fixes:**
- fix(scripts): CDPATH bug in shell scripts - prevents failures when
  users have CDPATH environment variable set (from spec-kit PR github#1039)

**Features:**
- feat(commands): add handoffs for workflow transitions - Copilot users
  see buttons, Claude users see suggested next steps
- feat(agents): add research-assistant and research-reviewer agents
  - research-assistant: Full SRD workflow orchestrator
  - research-reviewer: Quality assurance specialist
- feat(cli): add `research version` command
- feat(release): update script to deploy agents to correct locations
  - Claude: .claude/agents/*.md
  - Copilot: .github/agents/*.agent.md with companion .prompt.md
  - Cursor: .cursor/agents/*.md

**Documentation:**
- docs: add upgrade guide (docs/upgrade.md)
- docs: update AGENTS.md with agents section
- docs: update README.md with Research Agents section
- docs: update the-fork.md with sync history and examples

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

4 participants