Skip to content

Point this repository at its new home in acyclic-labs/sdk - #25

Open
ramstar3000 wants to merge 1 commit into
mainfrom
retire
Open

ramstar3000 wants to merge 1 commit into
mainfrom
retire

Conversation

@ramstar3000

@ramstar3000 ramstar3000 commented Sep 19, 2026

Copy link
Copy Markdown
Contributor

The plugin lives in acyclic-labs/sdk as plugin/ now (sdk #100), qualified and released there. This PR is the archive step:

  • README.md: pointer to the new location, install commands, where to file issues.
  • scripts/install.sh: a shim that runs the installer from the sdk repository, so the old curl one-liner keeps working.
  • .github/workflows/{ci,release}.yml removed: nothing here can publish the same npm packages twice. The product-name guard goes with the workflow that ran it.
  • CHANGELOG.md: names the imported commit.

After merging: archive the repository (gh repo archive acyclic-labs/graphcoder-plugin). Nothing in sdk depends on this repository any more.


Summary by cubic

Retires this repository: the plugin now lives in acyclic-labs/sdk as plugin/, where it is built, qualified, and released. This PR points the README at the new home, turns scripts/install.sh into a shim that runs the SDK repo's installer so the old curl one-liner keeps working, and removes the release workflow so nothing here can publish the same npm packages twice.

  • The CI workflow stays, so the product-name guard still runs and this final change is qualified; archiving the repo disables it afterwards.
  • CHANGELOG.md records the move and names the imported commit.
  • After merging, archive this repository; nothing in acyclic-labs/sdk depends on it.

Written for commit d954432. Summary will update on new commits.

Review in cubic

@greptile-apps

greptile-apps Bot commented Sep 19, 2026

Copy link
Copy Markdown

RetriggerConfidence Score: 5/5

The PR appears safe to merge, with only the two previously reported non-blocking documentation issues still outstanding.

Findings

  1. P2 Import commit is unavailable
  2. P2 Archive stub breaks references
Fix with agent prompt
### Issue 1
CHANGELOG.md:undefined-10
The changelog identifies `e82be0b` as the source-repository commit imported into the SDK, but that commit does not exist in this repository's history. Readers therefore cannot inspect or verify the archive boundary that this entry is intended to document.

### Issue 2
README.md:13-14
Replacing the README with this archive stub removes sections that the retained changelog still references, including “Retention and purge” and the configuration-layering contract. Those historical entries now direct readers to documentation that no longer exists; link them to the corresponding SDK documentation or preserve the referenced material.

---

For each issue above, determine whether it is valid and should be fixed. If so, fix it directly.
Summary

Retires this repository in favor of acyclic-labs/sdk/plugin, preserving the legacy installation URL through a forwarding shim and disabling duplicate releases.

  • Replaces the README with migration, installation, and issue-tracker pointers.
  • Records the imported repository boundary in the changelog.
  • Redirects the legacy installer to the SDK installer.
  • Removes the release workflow while retaining CI for the final archive transition.

Reviews (2) · Last reviewed commit: "Point this repository at its new home in..."

Comment thread CHANGELOG.md

This repository is archived. The plugin was imported into
[`acyclic-labs/sdk`](https://github.com/acyclic-labs/sdk) as `plugin/` at
commit `e82be0b` of this repository (sdk PR #100, 2026-09-19); everything

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 Import commit is unavailable

The changelog identifies e82be0b as the source-repository commit imported into the SDK, but that commit does not exist in this repository's history. Readers therefore cannot inspect or verify the archive boundary that this entry is intended to document.

Prompt To Fix With AI
This is a comment left during a code review.
Path: CHANGELOG.md
Line: 10

Comment:
**Import commit is unavailable**

The changelog identifies `e82be0b` as the source-repository commit imported into the SDK, but that commit does not exist in this repository's history. Readers therefore cannot inspect or verify the archive boundary that this entry is intended to document.

---

For each issue above, determine whether it is valid and should be fixed. If so, fix it directly.

Comment thread README.md
Comment on lines +13 to +14
This repository is archived. Its history up to the move is preserved here;
`CHANGELOG.md` names the commit that was imported.

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 Archive stub breaks references

Replacing the README with this archive stub removes sections that the retained changelog still references, including “Retention and purge” and the configuration-layering contract. Those historical entries now direct readers to documentation that no longer exists; link them to the corresponding SDK documentation or preserve the referenced material.

Prompt To Fix With AI
This is a comment left during a code review.
Path: README.md
Line: 13-14

Comment:
**Archive stub breaks references**

Replacing the README with this archive stub removes sections that the retained changelog still references, including “Retention and purge” and the configuration-layering contract. Those historical entries now direct readers to documentation that no longer exists; link them to the corresponding SDK documentation or preserve the referenced material.

---

For each issue above, determine whether it is valid and should be fixed. If so, fix it directly.

@greptile-apps

greptile-apps Bot commented Sep 19, 2026

Copy link
Copy Markdown

Comments Outside Diff

These findings sit on lines the diff does not cover, so they could not be posted inline. Each one leaves this list once its file changes.

  • P2 Deleted guard breaks local CI scripts/check-product-name.sh:43

    Deleting this guard leaves scripts/ci-local.sh invoking the nonexistent scripts/check-product-name.sh, so every local CI run records this required step as failed and exits unsuccessfully. CONTRIBUTING.md and packaging/npm/RELEASING.md also still refer to the deleted guard. Either retain it or update the remaining commands and documentation as part of the archive transition.

The plugin was imported into acyclic-labs/sdk as plugin/ (sdk PR #100); it
builds, qualifies and releases there now. This repository is being archived:
the README points at the new location, scripts/install.sh becomes a shim that
runs the installer from the sdk repository so the old curl one-liner keeps
working, and the release workflow is removed so nothing here can publish the
same npm packages twice. The CI workflow stays so this last change is
qualified like every other; archiving disables it afterwards.
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