Assets: OAP reader and mount set on IContentSource
Maintainers usually reply within 1 day
Nobody has claimed this yet.
Assessment
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Newbie friendliness
- 42/100
Research direction
Start with OapReader.cs:64-88, 270-313 and OapMountSet.cs:35-164, then inspect the IContentSource API and dependency issue #242. Run the OAP tests and the example-01 load path before changing these readers. Done means OAP reads and overlay enumeration use content sources, change detection uses identity, editor file watching still works, desktop export runs, and the acceptance criteria pass.
Written by the indexing model from the issue text.
Description
Current State
OapReader (OapReader.cs:64-88, 270-313) opens a FileStream and seeks per section; OapMountSet (OapMountSet.cs:35-164) uses File.Exists, Directory.EnumerateFiles for overlays and a FileInfo fingerprint checked on every read (EnsureUnchanged).
Tasks
-
OapReaderreads from anIContentSource -
OapMountSetmounts sources and detects changes by identity, not per-read stat - Overlays enumerate through the source
- Keep editor file-watching behavior
Acceptance Criteria
- OAP tests green; desktop export runs
- No
File.*in the OAP runtime path - No load-time regression on example-01
Related
- Android plan task A7 · seniority Mid · mergeable now (.NET 10)
- Depends on: #242
- Zstd for OAP: #133
- Platform roadmap: #100
- Dominant language
- C#
- Stars
- 1
- Forks
- 1
- Avg merge
- 1h 48m
- Merged PRs (30d)
- 30
Getting set up
- No Dockerfile or Docker Compose file
- Has a pull request template
- Read the contributing guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
More from MASS4ORG/Turian
-
c:docs p:high size:S t:docs
Difficulty 2/5 1-3 hours Newbie friendliness 64/100
Maintainers usually reply within 1 day
-
c:assets p:low size:XS t:task
Difficulty 2/5 1-3 hours Newbie friendliness 78/100
Maintainers usually reply within 1 day
-
c:engine c:scripting p:medium size:XL t:refactor
Difficulty 5/5 Over a week Newbie friendliness 25/100
Maintainers usually reply within 1 day
-
c:assets c:platform p:low size:M t:feature
Difficulty 4/5 3-5 days Newbie friendliness 42/100
Maintainers usually reply within 1 day
-
c:platform c:scripting p:low size:M t:task
Difficulty 5/5 Over a week Newbie friendliness 25/100
Maintainers usually reply within 1 day
Similar issues
-
bug next version
Difficulty 2/5 1-3 hours Newbie friendliness 68/100
Rem0o/FanControl.Releases#4220 · 1 comment ·
-
Difficulty 2/5 1-3 hours Newbie friendliness 78/100
nkdAgility/azure-devops-migration-tools#3217 ·
Maintainers usually reply within 1 day
-
Difficulty 2/5 1-3 hours Newbie friendliness 82/100
skoruba/Duende.IdentityServer.Admin#335 · 1 comment ·
Maintainers usually reply within 1 day
-
bug
Difficulty 2/5 1-3 hours Newbie friendliness 86/100
Maintainers usually reply within 1 day
-
question
Difficulty 2/5 1-3 hours Newbie friendliness 68/100