Skip to content
cccskills
FreePublished on GitHub

collector-owner-oid-resolution

Resolve collector ownership from persisted source records and fail closed on missing owner OIDs.

The English summary is being prepared. Showing the original description.

Run score Not tested

See how to install

Files included (1)

  • SKILL.md2.7 KB

SKILL.md (original)

Review the instructions the agent will receive before you install.

Context

Use this when a background collector has no authenticated user context but still needs to persist owned records. In JJGNet Broadcasting, the Round 1 fix is to resolve the owner OID from existing persisted source records and pass that OID all the way into the reader that materializes new content.

Patterns

  • Add a narrow GetCollectorOwnerOidAsync helper on the relevant manager/data-store pair.
  • Resolve the owner OID at the Function boundary before calling the reader.
  • If the owner OID is missing or blank, fail closed and stop before persistence.
  • Reader methods that materialize persistable domain models should require ownerOid and throw on blank values.
  • Regression coverage should stub GetCollectorOwnerOidAsync(...) with a distinct owner value and verify the collector passes that exact value into reader.GetAsync(...).
  • Regression coverage should verify SaveAsync(...) receives records whose CreatedByEntraOid is non-empty and matches the resolved owner.
  • Keep manually skipped integration tests compiling against the owner-aware overloads so repo-wide builds stay green.
  • Keep the scope narrow: do not broaden a collector-owner fix into OAuth/token-runtime redesign unless the issue explicitly asks for it.

Examples

  • src\JosephGuadagno.Broadcasting.Functions/Collectors/CollectorOwnerOidResolver.cs
  • src\JosephGuadagno.Broadcasting.Functions/Collectors/SyndicationFeed/LoadNewPosts.cs
  • src\JosephGuadagno.Broadcasting.Functions/Collectors/YouTube/LoadNewVideos.cs
  • src\JosephGuadagno.Broadcasting.Functions.Tests/Collectors/LoadNewPostsTests.cs
  • src\JosephGuadagno.Broadcasting.Functions.Tests/Collectors/LoadAllVideosTests.cs
  • src\JosephGuadagno.Broadcasting.SyndicationFeedReader.Tests/SyndicationFeedReaderOfflineTests.cs
  • src\JosephGuadagno.Broadcasting.YouTubeReader.Tests/YouTubeReaderFetchTests.cs

Anti-Patterns

  • Do not fall back to a global settings-level owner scaffold once source records already carry ownership.
  • Do not allow readers to construct persistable records with CreatedByEntraOid = string.Empty.
  • Do not hide missing-owner conditions by silently saving data with blank ownership.
  • Do not let skipped/manual integration tests drift onto removed overloads after the owner-aware interfaces become the contract.

Reviews

No reviews yet. Tell others what you thought.

Comments

No login needed. Share how it worked for you, ask a question, or note where it did not run.

No comments yet. Be the first to write one.

Comments are published right away. Do not include personal information such as email addresses or phone numbers. Up to 2 URLs.

0 / 1,000 characters

This form is protected by reCAPTCHA, and the Google Privacy Policy and Terms of Service apply.

More skills from this repository

Standard collaboration patterns for all squad agents — worktree awareness, decisions, cross-agent communication

The English summary is being prepared. Showing the original description.

jguadagno/jjgnet-broadcast18Updated Aug 4, 2026

Shared hard rules enforced across all squad agents

The English summary is being prepared. Showing the original description.

jguadagno/jjgnet-broadcast18Updated Aug 4, 2026

Comprehensive AI prompt engineering safety review and improvement prompt. Analyzes prompts for safety, bias, security vulnerabilities, and effectiveness while providing detailed improvement recommendations with extensive frameworks, testing methodologies, and educational content.

The English summary is being prepared. Showing the original description.

jguadagno/jjgnet-broadcast18Updated Aug 4, 2026

Migrate API controller scope checks to framework-enforced authorization policies while preserving ownership checks.

The English summary is being prepared. Showing the original description.

jguadagno/jjgnet-broadcast18Updated Aug 4, 2026

Review API DTO implementations for contract/domain separation, mapping patterns, and REST compliance

The English summary is being prepared. Showing the original description.

jguadagno/jjgnet-broadcast18Updated Aug 4, 2026

Register shared claims transformation and hierarchical role policies in the API without breaking existing scope enforcement.

The English summary is being prepared. Showing the original description.

jguadagno/jjgnet-broadcast18Updated Aug 4, 2026

See all skills by jguadagno

Report a problem with this skill