chore: restore repository policies and update refactor notices - #2383
Conversation
|
Claude Security Review: no high-confidence findings. (run) |
There was a problem hiding this comment.
AgentCore Harness Review
Verdict: Looks good
This PR restores standard OSS repo files (LICENSE, NOTICE, CODE_OF_CONDUCT.md, SECURITY.md, .github/pull_request_template.md, and the three .github/ISSUE_TEMPLATE/*.yml files) that are missing from the working branch. I verified every file in the diff is byte-identical (same blob hash) to what's already on main, so this is a pure restore with no risk.
No changes required.
Note (out of scope): the restored NOTICE is somewhat stale relative to the current package.json (missing bedrock-agentcore, winston, yaml, @tanstack/react-query, @opentelemetry/*, semver, lodash, etc., and lists some deps that are no longer present). Since the file is being restored verbatim from main, that drift already exists on main and isn't something this PR should fix — worth tracking separately.
|
Claude Security Review: no high-confidence findings. (run) |
|
Claude Security Review: no high-confidence findings. (run) |
|
Claude Security Review: no high-confidence findings. (run) |
This reverts commit 455a0ae.
|
Claude Security Review: no high-confidence findings. (run) |
|
Claude Security Review: no high-confidence findings. (run) |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## refactor #2383 +/- ##
=========================================
Coverage 97.25% 97.25%
=========================================
Files 613 613
Lines 41028 41028
=========================================
Hits 39902 39902
Misses 1126 1126 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
|
Thanks for the report, @codecov-commenter — feedback like this is exactly Would you mind opening a new issue so we can track it properly? If this is a security issue, please report it privately via |
Description
Restore repository policy, legal, and contribution-template files from
mainat805f3420e2069e83694ba12ef447d8495b883aef, with focused updates for the refactor:NOTICEin its original concise attribution-list format. Update current dependency entries, remove unused entries, and add short InkUI and Bun attributions. No generated dependency inventory or full license-text bundle.SECURITY.mdfrom Bedrock AgentCore CLI Starter Toolkit to AgentCore CLI. Keep all reporting instructions, contact information, and policy commitments unchanged frommain.LICENSE,CODE_OF_CONDUCT.md, and.github/ISSUE_TEMPLATE/config.ymlremain exact copies frommain.No changelog, Dependabot configuration, workflows, package metadata, or application code changes are included.
Related Issue
Follow-up repository-readiness work after #2380.
Documentation PR
Included in this PR.
Type of Change
Testing
bun run format:checkpassed across the repository.git diff --checkpassed.Application tests, typechecking, linting, and builds were not run because these changes affect documentation and repository templates only.
bun testbun run test:e2e, or explained why they are not applicablebun run typecheckbun run lint:checkbun run format:checkbun run buildsrc/assets/, I updated affected snapshots withbun test <test-file> --update-snapshotsand committed themChecklist
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the
terms of your choice.