Migrate from gpt-5.5 to claude-opus-4.8#63
Closed
meerakurup wants to merge 1 commit into
Closed
Conversation
nicholasdbrady
added a commit
that referenced
this pull request
Jul 8, 2026
…tributors (#67) * Fix fork PR preview deploy via workflow_run split, gated to forgebook-contributors Fork-based PRs always get a read-only GITHUB_TOKEN on pull_request events, regardless of the author's actual repo/team write access, causing the gh-pages deploy step to 403 (seen on PR #63 and #66). - preview.yml: trimmed to unprivileged build/validate/test; uploads the built site and PR metadata as artifacts instead of deploying directly. - preview-deploy.yml (new): workflow_run consumer with write access; only ever touches the pre-built static artifact, never PR code. Gates the actual gh-pages deploy + preview comment on forgebook-contributors team membership, via a maintainer-owned PAT (FORGEBOOK_TEAM_READ_TOKEN secret) since the default GITHUB_TOKEN can't read team membership. - preview-cleanup.yml (new): moved the PR-close cleanup job here on pull_request_target, since it has the same fork-token problem but is safe to run privileged (no PR code execution, just deletes a directory on the maintainer-controlled gh-pages branch). - CONTRIBUTING.md: note that auto-deployed previews require team membership; others still get build/test signal plus a comment. Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com> * Fix CodeQL code-injection findings: pass step outputs via env, not inline template github-script steps interpolated step outputs directly into the JS source via ${{ }} template expressions, which CodeQL flags as a code-injection risk since the value flows from a workflow_run artifact. Switched all occurrences (pr_number and author) to pass through env: and read via process.env instead, which is the standard mitigation. Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com> --------- Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
Adds a new Forgebook recipe for migrating a Chat Completions workload from
gpt-5.5toclaude-opus-4-8on Microsoft Foundry.The notebook teaches a side-by-side migration pattern: keep the existing GPT-5.5 path live, add Claude behind the same adapter contract, compare both providers with eval gates, then promote only after schema/tool parity, structured output retry rate, quality, and latency checks pass.
New content and contributor registration:
meerakurupas a new author inauthors.yaml, including name, title, avatar, and GitHub profile.registry.yamlwith slug, path, title, description, date, author, and allowed tags.notebooks/migrate-gpt-5-5-to-claude-opus-4-8.ipynb.Review focus
Please focus review on:
Conversation/Responsecontract is a reasonable reusable pattern..envconfiguration, quota assumptions, and common error recovery.Checklist
Notebook contributions
Skip if this PR only touches site code.
notebooks/directoryregistry.yamlwith all required fields (slug,path,title,authors)authors.yaml(with at leastnameandtitle)tags.yamlcd scripts && npx tsx validate-registry.tsnotebooks/media/<slug>/with descriptive alt textnotebooks/data/<slug>/when applicableSite changes
Skip if this PR only adds a notebook.
cd site && npm run buildcd site && npx astro check