Repository navigation
Description
Student: Aniket Manna
Task: R51-T036 · 6 of 10
Repository: https://github.com/S-AI-Projects/javascript-algorithms
Assigned working branch: student/e26b07f0860
Goal
Work on src/algorithms/search/linear-search/linearSearch.js. Refactor one coherent calculation in linearSearch while preserving its behavior for the fixture. Name the particular state or operation made clearer in the PR. Fixture: Search [2, 4, 6, 8] for 2, 8 and 5, then search a singleton and an empty collection if allowed. Use the source miss sentinel and duplicate-index contract.
Steps
- Read src/algorithms/search/linear-search/linearSearch.js, especially
linearSearchnear line 11. Record its public inputs, result, mutation behavior and any external dependencies before changing it. - First capture behavior in tests; identify the smallest contiguous responsibility that can be named clearly.
- Extract or simplify that responsibility without changing algorithm, return type, ordering, side effects or public arguments.
- Implement work/R51-T036/check.py as a focused runner that executes the new checks and propagates failures. Native check to incorporate: run the repository test script in non-watch mode after installing from its lockfile; for browser-only code use a browser test instead of plain Node. Record the actual command and nonzero test count.
- Create the checks specified below and run
python3 work/R51-T036/check.py. Save inputs, expected versus actual results and tool versions in work/R51-T036/; a check must exit non-zero when an assertion fails. - Commit the repository change on branch R51-T036; prepare a PR to the organisation fork explaining the result and linking the checks. Keep each task independently reviewable. Remove work/R51-T036/ from a later upstream branch.
Done when
☐ The checked behavior is tied to linearSearch in src/algorithms/search/linear-search/linearSearch.js; every assertion has an explicit expected value.
☐ Search [2, 4, 6, 8] for 2, 8 and 5, then search a singleton and an empty collection if allowed. Use the source miss sentinel and duplicate-index contract.
☐ Existing supported behavior and the public interface are preserved unless a failing regression demonstrates and documents a defect.
☐ The before/after tests agree and the diff contains one explained readability improvement.
☐ The change is not a repository-wide formatting pass or an unmeasured optimization.
Additional task guidance
Additional curated task guidance
Item:
src/algorithms/search/linear-search
Task type:
Focused refactoring
Code change (goes into the PR):
src/algorithms/search/linear-search/linearSearch.js: one localized refactor plus regression tests that lock down the fixture and the public contract.
Automated check:
python3 work/R51-T036/check.py
Prerequisites:
Node matching project CI, the project lockfile dependencies, and Python 3.
Upstream PR rules:
Keep the contribution scoped to the named item and follow the linked contribution instructions and repository CI. Submit coursework to the organisation fork first; keep work/ evidence out of any later upstream PR. Upstream acceptance is not a grading requirement. Policy: https://github.com/trekhleb/javascript-algorithms/blob/85293e3e2b88f4d2ce330d956b139cf628aa1e82/CONTRIBUTING.md
Change:
Replaces the shared task template with an item-specific contribution brief.
PR destination:
Upstream candidate after review
Submission checklist
- Complete this task and its checks.
- Add the required evidence and link your pull request here.
- Select this S-AI-Projects fork as the PR base repository.
- Request instructor review.
Keep this PR focused on this task. Do not submit assignment PRs to the upstream repository.
Activity