forked from farhoodlabs/paperclip
fix: remove GitHub source pinning warning from company import
We don't support regular updates to agents from GitHub sources yet, so the "not pinned to a commit SHA" warning is misleading and unnecessary. Co-Authored-By: Paperclip <noreply@paperclip.ing> Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
This commit is contained in:
@@ -1284,9 +1284,6 @@ function buildManifestFromPackageFiles(
|
|||||||
};
|
};
|
||||||
}
|
}
|
||||||
|
|
||||||
function isGitCommitRef(value: string) {
|
|
||||||
return /^[0-9a-f]{40}$/i.test(value.trim());
|
|
||||||
}
|
|
||||||
|
|
||||||
function parseGitHubSourceUrl(rawUrl: string) {
|
function parseGitHubSourceUrl(rawUrl: string) {
|
||||||
const url = new URL(rawUrl);
|
const url = new URL(rawUrl);
|
||||||
@@ -1413,9 +1410,6 @@ export function companyPortabilityService(db: Db) {
|
|||||||
const parsed = parseGitHubSourceUrl(source.url);
|
const parsed = parseGitHubSourceUrl(source.url);
|
||||||
let ref = parsed.ref;
|
let ref = parsed.ref;
|
||||||
const warnings: string[] = [];
|
const warnings: string[] = [];
|
||||||
if (!isGitCommitRef(ref)) {
|
|
||||||
warnings.push("GitHub source is not pinned to a commit SHA; imports may drift if the ref changes.");
|
|
||||||
}
|
|
||||||
const companyRelativePath = parsed.companyPath === "COMPANY.md"
|
const companyRelativePath = parsed.companyPath === "COMPANY.md"
|
||||||
? [parsed.basePath, "COMPANY.md"].filter(Boolean).join("/")
|
? [parsed.basePath, "COMPANY.md"].filter(Boolean).join("/")
|
||||||
: parsed.companyPath;
|
: parsed.companyPath;
|
||||||
|
|||||||
Reference in New Issue
Block a user