Skip to content

feat: CMS - Switch to copying files instead of hard-linking#541

Merged
zastrowm merged 1 commit intostrands-agents:workstream/cms-migrationfrom
zastrowm:switch_to_direct_copy
Feb 12, 2026
Merged

feat: CMS - Switch to copying files instead of hard-linking#541
zastrowm merged 1 commit intostrands-agents:workstream/cms-migrationfrom
zastrowm:switch_to_direct_copy

Conversation

@zastrowm
Copy link
Copy Markdown
Member

Description

Note: This is targeting the workstream/cms-migration branch.

Previously we used a hard-link to include files in starlight/astro so that mkdocs and starlight would continue to work together - however, this is now slowing down progress in generating api documentation (Python & TypeScript). So switch to doing a straight copy to the directory.

Changes in this PR:

  • Copy the relevant files from /docs -> src/content/docs which is where starlight needs them; previously we used a directory link
  • update-docs.ts to account for this; as noted before, this is a script that will be used to do the final migration and so the contents are not to be tightly reviewed unless you see something obvious. The final output is the more important part.

Related Issues

#441

Type of Change

  • Content update/revision

Checklist

  • I have read the CONTRIBUTING document
  • My changes follow the project's documentation style
  • I have tested the documentation locally using mkdocs serve
  • Links in the documentation are valid and working

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Previously we used a hard-link to include files in astro so that mkdocs and starlight would continue to work together - however, this is now slowing down progress in generating api documentation (Python & TypeScript).

Switch to doing a straight copy to the directory.
@github-actions
Copy link
Copy Markdown
Contributor

Documentation Preview Ready

Your documentation preview has been successfully deployed!

Preview URL: https://d3ehv1nix5p99z.cloudfront.net/pr-cms-541/user-guide/quickstart/overview/

Updated at: 2026-02-11T19:30:17.233Z

Comment thread scripts/update-docs.ts
Comment thread scripts/update-docs.ts
@mehtarac
Copy link
Copy Markdown
Member

/strands review

@github-actions
Copy link
Copy Markdown
Contributor

Assessment: Approve

This PR cleanly accomplishes the goal of switching from hard-links to direct file copying for the CMS migration. The path updates across configuration files are consistent and well-coordinated.

Review Notes
  • Config Changes: Logo paths, content base directory, and revert command all correctly updated to reflect the new file structure
  • Migration Script: The additions to update-docs.ts for copying TS, Python, and asset files are appropriate for one-time migration tooling
  • Symlink Removal: The removal of src/content/docs symlink and its replacement with direct file copying aligns with the stated goal of unblocking API documentation generation

The existing thread about code duplication in the walk functions has been appropriately addressed - given this is throwaway migration tooling, the pragmatic choice to not over-engineer it is reasonable.

@zastrowm zastrowm enabled auto-merge (squash) February 12, 2026 20:19
@zastrowm zastrowm merged commit 02c029d into strands-agents:workstream/cms-migration Feb 12, 2026
4 of 6 checks passed
This was referenced Feb 17, 2026
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.

3 participants